-
Notifications
You must be signed in to change notification settings - Fork 527
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
Feature/adding jwt token validation #178
Closed
eknowlton
wants to merge
34
commits into
mpolden:master
from
levelrmm:feature/adding-jwt-token-validation
Closed
Feature/adding jwt token validation #178
eknowlton
wants to merge
34
commits into
mpolden:master
from
levelrmm:feature/adding-jwt-token-validation
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adding IPStack
- IsDaylightSavings - IsProxy - IsTOR - IsCrawler - CrawlerName - CrawlerType - ThreatLevel - ThreatTypes
Added extra IP Stack fields
…extra-ipstack-fields
Adding More IP Stack Fields
- [x] Pipeline to update binary on server when changes are pushed - [x] Added config option for Cache TTL in Seconds in `/etc/echoip/config.toml - [x] Updated Readme for Cache Options - [x] Added `Null` cache for optional no use of cache - [x] Adding Redis Cache - [x] Moving configuration to file config - [x] Automatic Release for amd64 linux - [x] Adding automatic deployment - Added `LICENSE` file to the release file
Caching in Redis
I didn't want to kep the API Key in a file because it could be exposed so I thought a environment variable would be good. Might of got carried away and didn't need to put all config options in environment variables. commit ba35fa7 Author: Ethan Knowlton <ethan@dubtown.dev> Date: Sat Oct 7 22:35:31 2023 -0400 removed debug statement commit 3808cc2 Author: Ethan Knowlton <ethan@dubtown.dev> Date: Sat Oct 7 22:32:27 2023 -0400 Could Cause Problems Shouldn't need to run make as super user, but usually you would run `make install`. I could add this to the install step and remove the install from all step but it could mess up publishing to ip.level.io. Just going to remove it for now. commit aa0cb39 Author: Ethan Knowlton <ethan@dubtown.dev> Date: Sat Oct 7 20:05:14 2023 -0400 adding env variables to readme
…riables-for-docker Enable use of environment variables for docker
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added JWT Token Authentication
JWT Auth disabled by default
Added
Jwt
section to configI at some point put the usage statistics per
sub
from the JWT token inside redis, with the number of API calls but that didn't seem like the right place. I can push that or do something else if you have a idea of where it would be more useful.