fix(indexworker): remove pg_trgm extension

[fadymak]

Removes dependency on the pg_trgm extension in the indexworker. This simplifies the rollout process related to extension creation and permissions at the cost of more restricted query patterns (i.e.: no contains queries on name and email %term%.

[coveralls]

Pull Request Test Coverage Report for Build 20342665820

Details

• 13 of 15(86.67%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 68.607%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/indexworker/indexworker.go	13	15	86.67%

Totals
Change from base Build 20307749740:	0.07%
Covered Lines:	14636
Relevant Lines:	21333

---

💛 - Coveralls

[cstockton]

I noticed we parse url twice, once conditionally, a second time non-conditionally without checking errors. Does this fail the same way for all input states?

auth/internal/indexworker/indexworker.go

Lines 36 to 50 in 84dcb02

	ifconfig.DB.Driver==""&&config.DB.URL!="" {
	u, err:=url.Parse(config.DB.URL)
	iferr!=nil {
	le.Fatalf("Error parsing db connection url: %+v", err)
	}
	config.DB.Driver=u.Scheme
	}
	u, _:=url.Parse(config.DB.URL)
	processedUrl:=config.DB.URL
	iflen(u.Query()) !=0 {
	processedUrl=fmt.Sprintf("%s&application_name=auth_index_worker", processedUrl)
	} else {
	processedUrl=fmt.Sprintf("%s?application_name=auth_index_worker", processedUrl)
	}

[fadymak]

Removes dependency on the pg_trgm extension in the indexworker. This simplifies the rollout process related to extension creation and permissions at the cost of more restricted query patterns (i.e.: no contains queries on name and email %term%.

[coveralls]

Pull Request Test Coverage Report for Build 20342665820

Details

• 13 of 15(86.67%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 68.607%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/indexworker/indexworker.go	13	15	86.67%

Totals
Change from base Build 20307749740:	0.07%
Covered Lines:	14636
Relevant Lines:	21333

---

💛 - Coveralls

[cstockton]

I noticed we parse url twice, once conditionally, a second time non-conditionally without checking errors. Does this fail the same way for all input states?

auth/internal/indexworker/indexworker.go

Lines 36 to 50 in 84dcb02

	ifconfig.DB.Driver==""&&config.DB.URL!="" {
	u, err:=url.Parse(config.DB.URL)
	iferr!=nil {
	le.Fatalf("Error parsing db connection url: %+v", err)
	}
	config.DB.Driver=u.Scheme
	}
	u, _:=url.Parse(config.DB.URL)
	processedUrl:=config.DB.URL
	iflen(u.Query()) !=0 {
	processedUrl=fmt.Sprintf("%s&application_name=auth_index_worker", processedUrl)
	} else {
	processedUrl=fmt.Sprintf("%s?application_name=auth_index_worker", processedUrl)
	}

[fadymak]

Removes dependency on the pg_trgm extension in the indexworker. This simplifies the rollout process related to extension creation and permissions at the cost of more restricted query patterns (i.e.: no contains queries on name and email %term%.

[coveralls]

Pull Request Test Coverage Report for Build 20342665820

Details

• 13 of 15(86.67%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 68.607%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/indexworker/indexworker.go	13	15	86.67%

Totals
Change from base Build 20307749740:	0.07%
Covered Lines:	14636
Relevant Lines:	21333

---

💛 - Coveralls

[cstockton]

I noticed we parse url twice, once conditionally, a second time non-conditionally without checking errors. Does this fail the same way for all input states?

auth/internal/indexworker/indexworker.go

Lines 36 to 50 in 84dcb02

	ifconfig.DB.Driver==""&&config.DB.URL!="" {
	u, err:=url.Parse(config.DB.URL)
	iferr!=nil {
	le.Fatalf("Error parsing db connection url: %+v", err)
	}
	config.DB.Driver=u.Scheme
	}
	u, _:=url.Parse(config.DB.URL)
	processedUrl:=config.DB.URL
	iflen(u.Query()) !=0 {
	processedUrl=fmt.Sprintf("%s&application_name=auth_index_worker", processedUrl)
	} else {
	processedUrl=fmt.Sprintf("%s?application_name=auth_index_worker", processedUrl)
	}

[fadymak]

Removes dependency on the pg_trgm extension in the indexworker. This simplifies the rollout process related to extension creation and permissions at the cost of more restricted query patterns (i.e.: no contains queries on name and email %term%.

[coveralls]

Pull Request Test Coverage Report for Build 20342665820

Details

• 13 of 15(86.67%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 68.607%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/indexworker/indexworker.go	13	15	86.67%

Totals
Change from base Build 20307749740:	0.07%
Covered Lines:	14636
Relevant Lines:	21333

---

💛 - Coveralls

[cstockton]

I noticed we parse url twice, once conditionally, a second time non-conditionally without checking errors. Does this fail the same way for all input states?

auth/internal/indexworker/indexworker.go

Lines 36 to 50 in 84dcb02

	ifconfig.DB.Driver==""&&config.DB.URL!="" {
	u, err:=url.Parse(config.DB.URL)
	iferr!=nil {
	le.Fatalf("Error parsing db connection url: %+v", err)
	}
	config.DB.Driver=u.Scheme
	}
	u, _:=url.Parse(config.DB.URL)
	processedUrl:=config.DB.URL
	iflen(u.Query()) !=0 {
	processedUrl=fmt.Sprintf("%s&application_name=auth_index_worker", processedUrl)
	} else {
	processedUrl=fmt.Sprintf("%s?application_name=auth_index_worker", processedUrl)
	}

[fadymak]

Removes dependency on the pg_trgm extension in the indexworker. This simplifies the rollout process related to extension creation and permissions at the cost of more restricted query patterns (i.e.: no contains queries on name and email %term%.

[coveralls]

Pull Request Test Coverage Report for Build 20342665820

Details

• 13 of 15(86.67%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 68.607%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/indexworker/indexworker.go	13	15	86.67%

Totals
Change from base Build 20307749740:	0.07%
Covered Lines:	14636
Relevant Lines:	21333

---

💛 - Coveralls

[cstockton]

I noticed we parse url twice, once conditionally, a second time non-conditionally without checking errors. Does this fail the same way for all input states?

auth/internal/indexworker/indexworker.go

Lines 36 to 50 in 84dcb02

	ifconfig.DB.Driver==""&&config.DB.URL!="" {
	u, err:=url.Parse(config.DB.URL)
	iferr!=nil {
	le.Fatalf("Error parsing db connection url: %+v", err)
	}
	config.DB.Driver=u.Scheme
	}
	u, _:=url.Parse(config.DB.URL)
	processedUrl:=config.DB.URL
	iflen(u.Query()) !=0 {
	processedUrl=fmt.Sprintf("%s&application_name=auth_index_worker", processedUrl)
	} else {
	processedUrl=fmt.Sprintf("%s?application_name=auth_index_worker", processedUrl)
	}

[fadymak]

Removes dependency on the pg_trgm extension in the indexworker. This simplifies the rollout process related to extension creation and permissions at the cost of more restricted query patterns (i.e.: no contains queries on name and email %term%.

[coveralls]

Pull Request Test Coverage Report for Build 20342665820

Details

• 13 of 15(86.67%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 68.607%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/indexworker/indexworker.go	13	15	86.67%

Totals
Change from base Build 20307749740:	0.07%
Covered Lines:	14636
Relevant Lines:	21333

---

💛 - Coveralls

[cstockton]

I noticed we parse url twice, once conditionally, a second time non-conditionally without checking errors. Does this fail the same way for all input states?

auth/internal/indexworker/indexworker.go

Lines 36 to 50 in 84dcb02

	ifconfig.DB.Driver==""&&config.DB.URL!="" {
	u, err:=url.Parse(config.DB.URL)
	iferr!=nil {
	le.Fatalf("Error parsing db connection url: %+v", err)
	}
	config.DB.Driver=u.Scheme
	}
	u, _:=url.Parse(config.DB.URL)
	processedUrl:=config.DB.URL
	iflen(u.Query()) !=0 {
	processedUrl=fmt.Sprintf("%s&application_name=auth_index_worker", processedUrl)
	} else {
	processedUrl=fmt.Sprintf("%s?application_name=auth_index_worker", processedUrl)
	}

[fadymak]

Removes dependency on the pg_trgm extension in the indexworker. This simplifies the rollout process related to extension creation and permissions at the cost of more restricted query patterns (i.e.: no contains queries on name and email %term%.

[coveralls]

Pull Request Test Coverage Report for Build 20342665820

Details

• 13 of 15(86.67%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 68.607%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/indexworker/indexworker.go	13	15	86.67%

Totals
Change from base Build 20307749740:	0.07%
Covered Lines:	14636
Relevant Lines:	21333

---

💛 - Coveralls

[cstockton]

I noticed we parse url twice, once conditionally, a second time non-conditionally without checking errors. Does this fail the same way for all input states?

auth/internal/indexworker/indexworker.go

Lines 36 to 50 in 84dcb02

	ifconfig.DB.Driver==""&&config.DB.URL!="" {
	u, err:=url.Parse(config.DB.URL)
	iferr!=nil {
	le.Fatalf("Error parsing db connection url: %+v", err)
	}
	config.DB.Driver=u.Scheme
	}
	u, _:=url.Parse(config.DB.URL)
	processedUrl:=config.DB.URL
	iflen(u.Query()) !=0 {
	processedUrl=fmt.Sprintf("%s&application_name=auth_index_worker", processedUrl)
	} else {
	processedUrl=fmt.Sprintf("%s?application_name=auth_index_worker", processedUrl)
	}

[fadymak]

Removes dependency on the pg_trgm extension in the indexworker. This simplifies the rollout process related to extension creation and permissions at the cost of more restricted query patterns (i.e.: no contains queries on name and email %term%.

[coveralls]

Pull Request Test Coverage Report for Build 20342665820

Details

• 13 of 15(86.67%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 68.607%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
internal/indexworker/indexworker.go	13	15	86.67%

Totals
Change from base Build 20307749740:	0.07%
Covered Lines:	14636
Relevant Lines:	21333

---

💛 - Coveralls

[cstockton]

I noticed we parse url twice, once conditionally, a second time non-conditionally without checking errors. Does this fail the same way for all input states?

auth/internal/indexworker/indexworker.go

Lines 36 to 50 in 84dcb02

	ifconfig.DB.Driver==""&&config.DB.URL!="" {
	u, err:=url.Parse(config.DB.URL)
	iferr!=nil {
	le.Fatalf("Error parsing db connection url: %+v", err)
	}
	config.DB.Driver=u.Scheme
	}
	u, _:=url.Parse(config.DB.URL)
	processedUrl:=config.DB.URL
	iflen(u.Query()) !=0 {
	processedUrl=fmt.Sprintf("%s&application_name=auth_index_worker", processedUrl)
	} else {
	processedUrl=fmt.Sprintf("%s?application_name=auth_index_worker", processedUrl)
	}