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

Add HTTP User Detail extension #171

Merged
merged 1 commit into from
May 31, 2024
Merged

Conversation

hannesdejager
Copy link
Collaborator

@hannesdejager hannesdejager commented Mar 1, 2024

This adds functionality to unFTP that allows unFTP to obtain user
detail over HTTP in addition to the already existing JSON file.

It uses the extact same format as the JSON file functionality. The
'usr-http-url' command line arguments activate this feature. You
pass it a base URL, unFTP appends a username to it and performs
a GET request to the URL. The HTTP server should respond with a
200 OK and JSON body containing an array of users which should
at least contain the requested user's details.

Later on we can support diffent HTTP verbs and sending the
username via an HTTP header instead of the URL path or Post
body. For now I'm just keeping it simple.

@hannesdejager hannesdejager marked this pull request as draft March 1, 2024 13:44
@hannesdejager hannesdejager force-pushed the hannes/usrdetail-http branch 4 times, most recently from 7233987 to ccc48f1 Compare March 1, 2024 15:32
@hannesdejager hannesdejager requested a review from robklg May 8, 2024 20:54
@hannesdejager hannesdejager self-assigned this May 8, 2024
@hannesdejager hannesdejager marked this pull request as ready for review May 8, 2024 21:01
This adds functionality to unFTP that allows unFTP to obtain user
detail over HTTP in addition to the already existing JSON file.

It uses the extact same format as the JSON file functionality. The
'usr-http-url' command line arguments activate this feature. You
pass it a base URL, unFTP appends a username to it and performs
a GET request to the URL. The HTTP server should respond with a
200 OK and JSON body containing an array of users which should
at least contain the requested user's details.

Later on we can support diffent HTTP verbs and sending the
username via an HTTP header instead of the URL path or Post
body. For now I'm just keeping it simple.
@hannesdejager hannesdejager merged commit 02c6c4e into master May 31, 2024
10 of 11 checks passed
@hannesdejager hannesdejager deleted the hannes/usrdetail-http branch June 20, 2024 20:53
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.

2 participants