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

Added Enqueue Route #13

Merged
merged 8 commits into from
Jun 14, 2018
Merged

Conversation

almacken
Copy link
Contributor

@almacken almacken commented Jun 6, 2018

Note - Did not add any authentication protocols, so this route is currently open to all users.

Note that it requires features to be mapped to feature descriptions to return them, and currently no features are mapped so no features are returned.
@ghost ghost assigned almacken Jun 6, 2018
@ghost ghost added the review label Jun 6, 2018
This actor forwards requests to another actor, and blocks IPs for a certain amount of time if they make too many requests within a given window.
It is not currently functioning, as there is not registration feature in this version, but it should work with a minor modification to the retrieve route.
@almacken
Copy link
Contributor Author

almacken commented Jun 7, 2018

I've added support for limiting requests from unrecognized users to this branch.
This pull request should now contribute to the features described in issues #7 and #8.
The only feature missing from this branch is some authentication feature. Once that is implemented, issues #7 and #8 can be resolved by simply changing the "pass" directives marked with TODOs into directives that require authentication.

@bhermann bhermann merged commit a65271d into delphi-hub:develop Jun 14, 2018
@ghost ghost removed the review label Jun 14, 2018
@almacken almacken deleted the feature/enqueue branch June 14, 2018 20:37
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