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

Bkalashi.basic auth support #256

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

bkalashian
Copy link

Note: Filling out this template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the
maintainer's discretion.

Description of the Change

Support (optional) Basic Authentication for retrieval of data.

Why Should This Be In Core?

Adds another layer of security for developers.

Benefits

Security benefit of allowing only authorized requests to attain requested data.

Potential Drawbacks

None, this is an optional parameter.

Applicable Issues

Utilized in LandSAR, Land Search and Rescue, a Search and Rescue project developed by JPRA and Raytheon BBN.

@bkalashian
Copy link
Author

[Bump], I just wanted to check on status of this PR. Thank you.

@crockct
Copy link

crockct commented Apr 21, 2023

This PR resolves issue #264

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants