Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

horizon: Modify tx submission system to work with RO database #4418

Merged
merged 16 commits into from
Jun 14, 2022
Merged

horizon: Modify tx submission system to work with RO database #4418

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
9b5750f
Modify tx submission system to work with RO database
2opremio May 31, 2022
14aedc8
Remove leftover mock
2opremio May 31, 2022
2d2ffb9
4418: fixing test failures
sreuland Jun 3, 2022
fb76bcd
4418: fixing issues with scenarios in unit tests
sreuland Jun 3, 2022
6683554
#4418: fixed integration tests related to txsub
sreuland Jun 4, 2022
f78dea2
Revert leftover (debug?) changes
2opremio Jun 6, 2022
2e3cc6f
#4418: repaired a few other tests that weren't mocking out Transactio…
sreuland Jun 6, 2022
18719f8
updated comments on tmp sql migration to use CREATE..AS..SELECT per a…
sreuland Jun 6, 2022
f18fce9
#4418: added DSN for read-only role on db connections in unit tests, …
sreuland Jun 8, 2022
dfaf5b7
Add schema match test
2opremio Jun 7, 2022
55f4b0a
#4418: changed integration tests to spin up one horizon web node with…
sreuland Jun 9, 2022
84b48f9
#4418: add default pwd for pg read only user on db connections
sreuland Jun 9, 2022
4c1ba36
rebased to 2.18.0, renamed migration to 56_txsub_read_only.sql
sreuland Jun 10, 2022
1488013
changed log level on filtered tx's to debug, added more detail on thr…
sreuland Jun 14, 2022
9b156d1
added test coverage for tx result query err conditions
sreuland Jun 14, 2022
ae3f81e
fixed broken test coverage on mock calls
sreuland Jun 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
#4418: add default pwd for pg read only user on db connections
  • Loading branch information
@sreuland
sreuland committed Jun 10, 2022
commit 84b48f97622c9822a132b7bbdcaeb63ce8128d48
10 changes: 0 additions & 10 deletions services/horizon/internal/test/integration/integration.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -188,16 +188,6 @@ func (i *Test) configureCaptiveCore() {
}
}

func (i *Test) getWebParameter(argName, envName string) string {
if value, ok := i.config.HorizonEnvironment[envName]; ok {
return value
}
if value, ok := i.config.HorizonWebParameters[argName]; ok {
return value
}
return ""
}

func (i *Test) getIngestParameter(argName, envName string) string {
if value, ok := i.config.HorizonEnvironment[envName]; ok {
return value
Expand Down
4 changes: 2 additions & 2 deletions support/db/dbtest/db.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ func checkReadOnly(t testing.TB, DSN string) {
require.NoError(t, err)

if !rows.Next() {
_, err = tx.Exec("CREATE ROLE user_ro LOGIN;")
_, err = tx.Exec("CREATE ROLE user_ro WITH LOGIN PASSWORD 'user_ro';")
require.NoError(t, err)
}

Expand All @@ -153,7 +153,7 @@ func Postgres(t testing.TB) *DB {

postgresDSN := fmt.Sprintf("postgres:https://%s@localhost/?sslmode=disable", pgUser)
result.DSN = fmt.Sprintf("postgres:https://%s@localhost/%s?sslmode=disable&timezone=UTC", pgUser, result.dbName)
result.RO_DSN = fmt.Sprintf("postgres:https://%s@localhost/%s?sslmode=disable&timezone=UTC", "user_ro", result.dbName)
result.RO_DSN = fmt.Sprintf("postgres:https://%s:%s@localhost/%s?sslmode=disable&timezone=UTC", "user_ro", "user_ro", result.dbName)

execStatement(t, fmt.Sprintf("CREATE DATABASE %s;", result.dbName), postgresDSN)
execStatement(t, fmt.Sprintf("GRANT CONNECT ON DATABASE %s TO PUBLIC;", result.dbName), postgresDSN)
Expand Down