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.

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

diff: audit #118

Closed
wants to merge 276 commits into from
Closed

diff: audit #118

wants to merge 276 commits into from

Conversation

zhongeric
Copy link
Collaborator

since 4b4d2e3

fix: give gouda-service permission to send order logs
feat: grant DS role access to S3
fix: add quoteId to responseJoi
chore: tweak fierhose log processor configs
feat: add filler to response json
fix: include quoteId in response Joi
chore: add timestamp related field
* disable Sev2 4XX and keep Sev3

* fix unit tests
feat: add synth-switch lambda
* add base sql and params support

* add validate configs func

* Add initial token price provider

* Add apollo, graphql provider

* Add logic to update synth repo

* remove unused var

* lowercase all addresses in configs

* change PI to boolean

* same

* remove apollo, comment out graphql provider for now

* yarn fix

* Fix update logic

* fix func call

* add TODOs

* Remove limit

* rename

* fix:prettier

* simplify

* lowercase reuslts from sql query

* add comment

* review

* fix logging

* Add parseKey test

* pass test

* remove token provider for now

* bump timeout to 10 min

* Fix bugs while testing, fix sql query

* Clean up code

* resole comments

* fix sql query and add basic unit test for validation

* formatting
@zhongeric zhongeric closed this Sep 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants