updated environment variables in README.md#28
Open
Tosinibikunle wants to merge 150 commits intoagstack:mainfrom
Open
updated environment variables in README.md#28Tosinibikunle wants to merge 150 commits intoagstack:mainfrom
Tosinibikunle wants to merge 150 commits intoagstack:mainfrom
Conversation
env vars changed to DB_NAME, DB_PASS and so on, and allow the use of DATABASE_URL to provide the full db as well to use a different engine than mysql. eg.: DATABASE_URL=postgres://gatekeeper_admin:somepassword@localhost:5432/gatekeeper
the cookie solution won't be a reliable thing in the end, since I noted that we will be getting lots of CORS issues, and even in browsers that this is still working, it is very likelly that it will stop on a near future. So Im using a "webhook" approach , passing back the token to the previous service on a specific url. then it's reponsability of that service to do what it needs (if it needs) on its own end.
This is the atual part of the reverse proxy view
Docs: Update API.md with latest endpoint details
Request data body
Code optimised
CORS header changes
testing agstack access
fixed some vulnerabilities
Eu funding readme
fixed CORS issue
JWT token lifetime reduced, and added in ENV, can be overridden
new admin module added. New API added for additional user info.
badges & bulk actions; CSV export; lock down token models; blob uploa…
arruda
requested changes
Oct 10, 2025
| | `JWT_ALG` | Algorithm used for JWT signing. | `HS256` | | ||
| | `JWT_ACCESS_TOKEN_MINUTES` | Access token expiration time (in minutes). | `60` | | ||
| | `JWT_REFRESH_TOKEN_DAYS` | Refresh token expiration time (in days). | `30` | | ||
| | `FARM_CALENDAR_API` | Base API URL for the Farm Calendar service. | `http://127.0.0.1:8002/api/` | |
Member
There was a problem hiding this comment.
@Tosinibikunle, thanks for your effort in this PR!
Could I ask you to just add back the '*' after the FARM_CALENDAR_API and FARM_CALENDAR_POST_AUTH in this great table that you've made? Apart from that this is looking good! =)
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
fixes #27