-
Notifications
You must be signed in to change notification settings - Fork 25
WIP: 🕸️ Bug / Support AWS OpenSearch Auth #1081
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
Draft
demariadaniel
wants to merge
23
commits into
main
Choose a base branch
from
bug/support-aws-auth
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
4685670
Add AWS auth setup & type corrections
demariadaniel 16daf53
Merge branch 'main' of https://github.com/overture-stack/arranger int…
demariadaniel bf529fc
Remove local logging
demariadaniel d636aac
Remove pnpm lock
demariadaniel da47b8b
Restore admin/utils.ts changes
demariadaniel d4edc0f
Use .gitignore for /admin
demariadaniel 7275919
Change setup for AWS Auth
demariadaniel eaa1762
Revise boilerplate comment
demariadaniel ab084cf
Auth config changes
demariadaniel 0ee6553
Remove aws-sdk
demariadaniel f74aa4a
Improve getOptions / createSearchClient
demariadaniel a61d27f
Error & Type improvements
demariadaniel eb2f820
Type cleanup
demariadaniel 17070fd
Add Readme docs for AWS OpenSearch
demariadaniel 897442d
Change key name to clientType, removed unused import
demariadaniel 439d760
Reinstall, Remove AWS SDK
demariadaniel 5fa7b18
Use StandardAuth config definition
demariadaniel 94db9a7
Use Type aliases
demariadaniel 7393589
Lowercase Aws names
demariadaniel e39af2a
Revert enableDebugging coercion
demariadaniel 7f995d3
Additional Capitalize Changes
demariadaniel b97e8f2
Standard Refresh type
demariadaniel ce4504d
Merge branch 'main' of https://github.com/overture-stack/arranger int…
demariadaniel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -16,4 +16,5 @@ lerna-debug.log | |
| !.env.test | ||
| !**/.env.schema | ||
|
|
||
| /configs | ||
| /configs | ||
| /modules/graphql-router/src/admin/** | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
12 changes: 5 additions & 7 deletions
12
modules/graphql-router/src/searchClient/createOpenSearchClient.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We aren't adding any code to consume these env variables, in fact we make no code changes to the server. Let's not add new env variables to our schema that are unused.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted get a start on documenting the Auth changes and new config values, then updated the client side without revisiting. Will revise.