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

HARMONY-1938: Remove fallback authentication functionality and related code #54

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

chris-durbin
Copy link
Contributor

@chris-durbin chris-durbin commented Dec 3, 2024

Jira Issue ID

HARMONY-1938

Description

In order to support local Harmony development without requiring an Earthdata Login application we needed to make some changes to no longer require OAUTH* environment variables with the EDL app information. As part of that change we're also removing the FALLBACK_AUTHN_ENABLED functionality that has not been used for the past few years which required an EDL client application in order to authenticate using a username and password.

Local Test Steps

Test along with the harmony-1938 branch in the main harmony repo. Make sure that services start correctly when OAUTH* environment variables are not set, and the services can continue to successfully download data.

I used this branch for building the opera-rtc-s1-browse image - https://github.com/chris-durbin/opera-rtc-s1-browse/tree/use-local-harmony-service-lib

  1. Clone the above repo and checkout the use-local-harmony-service-lib branch.
  2. Make sure you are on this harmony-1938 branch in the harmony-service-lib-py repo.
  3. run build-service
  4. Set the variables in your harmony .env:
OPERA_RTC_S1_BROWSE_IMAGE=asf/opera-rtc-reproject:latest
OPERA_RTC_S1_BROWSE_SERVICE_QUEUE_URLS='["asf/opera-rtc-reproject:latest,http://sqs.us-west-2.localhost.localstack.cloud:4566/000000000000/opera-rtc-s1-browse.fifo"]'
  1. Include opera-rtc-s1-browse in your LOCALLY_DEPLOYED_SERVICES.
  2. bin/restart-services

You can then submit a request: http://localhost:3000/C1259974840-ASF/ogc-api-coverages/1.0.0/collections/all/coverage/rangeset?granuleId=G1260014889-ASF&format=image%2Fpng&skipPreview=true

It will fail when it gets to HyBIG (unless you also update it to use the local service library changes), but you can verify that the service produced the correct outputs by looking at your local s3 artifacts bucket.

PR Acceptance Checklist

  • Acceptance criteria met
  • Tests added/updated (if needed) and passing
  • Documentation updated (if needed)

Copy link
Member

@ygliuvt ygliuvt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested successfully with locally built harmony-service-examples and harmony-1938 branch of the harmony project.

@chris-durbin chris-durbin merged commit 6240b6d into main Dec 9, 2024
12 checks passed
@chris-durbin chris-durbin deleted the harmony-1938 branch December 9, 2024 13:05
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.

3 participants