LG-7914 IPP PO Search: ArcGIS API Auth Strategy change#7223
Merged
allthesignals merged 2 commits intomainfrom Oct 26, 2022
Merged
LG-7914 IPP PO Search: ArcGIS API Auth Strategy change#7223allthesignals merged 2 commits intomainfrom
allthesignals merged 2 commits intomainfrom
Conversation
allthesignals
force-pushed
the
wmg/7914-auth-strategy
branch
2 times, most recently
from
0f49599 to
d21eabb
Compare
2 tasks
allthesignals
force-pushed
the
wmg/7914-auth-strategy
branch
from
d21eabb to
e566915
Compare
allthesignals
commented
Oct 26, 2022
allthesignals
commented
Oct 26, 2022
allthesignals
force-pushed
the
wmg/7914-auth-strategy
branch
7 times, most recently
from
669a736 to
afb5ffc
Compare
aduth
reviewed
Oct 26, 2022
allthesignals
force-pushed
the
wmg/7914-auth-strategy
branch
2 times, most recently
from
ec03a40 to
4aa122b
Compare
allthesignals
force-pushed
the
wmg/7914-auth-strategy
branch
from
4aa122b to
9335f3d
Compare
… strategy changed
allthesignals
force-pushed
the
wmg/7914-auth-strategy
branch
from
9335f3d to
50d66e4
Compare
Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
Merged
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
🎫 Ticket
Link to the relevant ticket.
🛠 Summary of changes
Previously, I was using a long-lived API Token I generated from the GSA-supplied ESRI-hosted ArcGIS API (developer dashboard UI).
Now, I am using the OAuth-style authorization strategy from the GSA-hosted GIS Enterprise Platform. Further, I am directing API calls to the GSA-hosted "portal" service.
This includes changes to the particular endpoint pathnames used.
Implements "dynamic_headers," borrowed from Proofer.
Adds new tests for implicit re-auth upon expiration.
📜 Testing Plan
Provide a checklist of steps to confirm the changes.
🚀 Notes for Deployment
We will need to update the environment variables for these new credentials.