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

Removes support for APIKEY #76

Merged
merged 1 commit into from
Jan 19, 2024
Merged

Removes support for APIKEY #76

merged 1 commit into from
Jan 19, 2024

Conversation

hankwallace
Copy link

@hankwallace hankwallace commented Jan 19, 2024 •
edited
Loading

Overview

  • Removes support for APIKEY. This is a step in updating our fork and in simplifying our build/deploy process for our hapi-fhir servers.

How it was tested

  • Tested by running the unit and postman tests locally.
  • Searched all task definitions to confirm that basic authentication was no longer used.

Checklist

  • The title contains a short meaningful summary
  • I have added a link to this PR in the Jira issue
  • I have described how this was tested
  • I have included screen shots for changes that affect the user interface
  • I have updated unit tests
  • I have run unit tests locally
  • I have updated documentation (including README)

@hankwallace hankwallace merged commit 5af64a0 into master Jan 19, 2024
5 checks passed
@hankwallace hankwallace deleted the remove-apikey branch January 20, 2024 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@semicolin semicolin semicolin approved these changes

@shubhamparikh927 shubhamparikh927 Awaiting requested review from shubhamparikh927

@ddjain ddjain Awaiting requested review from ddjain

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hankwallace @semicolin