Skip to content
This repository has been archived by the owner on Oct 14, 2023. It is now read-only.

basic auth implementation #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

basic auth implementation #31

wants to merge 1 commit into from

Conversation

21chanas3
Copy link
Contributor

adding basic authentication

@21chanas3
Copy link
Contributor Author

just to be safe

@ElioDiNino
Copy link
Member

How does the authentication work? Does it require specific login fields in requests to the program endpoints?

@ElioDiNino ElioDiNino added Priority: Medium Status: Review Needed Awaiting approval from a lead Type: Enhancement New feature or request Type: Security Security fix to be made labels Apr 2, 2023
@21chanas3
Copy link
Contributor Author

21chanas3 commented Apr 2, 2023 •
edited
Loading

yes, it simply requires a username and password (defined in .env) for all requests.

auth type is HTTP Basic and will require and “Authorization” header

@ElioDiNino
Copy link
Member

yes, it simply requires a username and password (defined in .env) for all requests.

auth type is HTTP Basic and will require and “Authorization” header

Ok cool, that's great. Is the header mentioned in the readme at all and do you know if integrated services (Frontend and MP Scripts) are ready for this change?

@21chanas3
Copy link
Contributor Author

Unsure, but will be updating the doc soon to reflect recent changes - I'll add a section about this on the Home page. Probably best to wait a week or so before merging this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@21chanas3 21chanas3

Labels
Priority: Medium Status: Review Needed Awaiting approval from a lead Type: Enhancement New feature or request Type: Security Security fix to be made
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@21chanas3 @ElioDiNino