Add oauth2_client options to app.ini.tmpl #28

Closed
two-first-names wants to merge 0 commits from two-first-names/gitea:feature/gitea-oauth2-client into master
First-time contributor
No description provided.
two-first-names added 1 commit 2022-09-19 15:55:44 +00:00
Add oauth2_client options to app.ini.tmpl
Some checks failed
continuous-integration/drone/pr Build is failing
9c591965d2
decentral1se requested changes 2022-09-19 16:11:18 +00:00
decentral1se left a comment
Owner

Nice! You'll need to follow up with a few changes:

  • bumping the app.ini.tmpl version in https://git.coopcloud.tech/coop-cloud/gitea/src/branch/master/abra.sh so that the config is upgraded when existing deployments roll out this new version (swarm configs are immutable)

  • adding the new env vars to https://git.coopcloud.tech/coop-cloud/gitea/src/branch/master/.env.sample so that folks know they can be config'd. if they're more optional, leave them commented out (don't forget to add GITEA_OAUTH2_CLIENT_ENABLED also)

  • thread the env vars through the environment: ... stanza so they are actually passed to the app itself in

    compose.yml Lines 18 to 34 in b757b806e4
    environment:
    - GITEA_ALLOW_ONLY_EXTERNAL_REGISTRATION
    - GITEA_APP_NAME
    - GITEA_AUTO_WATCH_NEW_REPOS
    - GITEA_DB_HOST="db:3306"
    - GITEA_DB_NAME=gitea
    - GITEA_DB_TYPE=mysql
    - GITEA_DB_USER=gitea
    - GITEA_DISABLE_REGISTRATION
    - GITEA_DOMAIN=${DOMAIN}
    - GITEA_ENABLE_NOTIFY_MAIL
    - GITEA_ENABLE_OPENID_SIGNIN
    - GITEA_ENABLE_OPENID_SIGNUP
    - GITEA_SMTP_MAILER_ENABLED
    - GITEA_SSH_PORT
    - GITEA_DISABLE_GRAVATAR
    - GITEA_ENABLE_FEDERATED_AVATAR
    (don't forget to add GITEA_OAUTH2_CLIENT_ENABLED also)

Nice! You'll need to follow up with a few changes: - bumping the `app.ini.tmpl` version in https://git.coopcloud.tech/coop-cloud/gitea/src/branch/master/abra.sh so that the config is upgraded when existing deployments roll out this new version (swarm configs are immutable) - adding the new env vars to https://git.coopcloud.tech/coop-cloud/gitea/src/branch/master/.env.sample so that folks know they can be config'd. if they're more optional, leave them commented out (don't forget to add `GITEA_OAUTH2_CLIENT_ENABLED` also) - thread the env vars through the `environment: ...` stanza so they are actually passed to the app itself in https://git.coopcloud.tech/coop-cloud/gitea/src/commit/b757b806e46308b90c3b1381b8fe19e02131de95/compose.yml#L18-L34 (don't forget to add `GITEA_OAUTH2_CLIENT_ENABLED` also)
Owner

heya @two-first-names, coming back to this - any updates?

heya @two-first-names, coming back to this - any updates?
Owner

Let's close this off for now, last comms 2 years ago.

Let's close this off for now, last comms 2 years ago.
decentral1se closed this pull request 2025-01-08 08:51:46 +00:00
Some checks failed
continuous-integration/drone/pr Build is failing

Pull request closed

Sign in to join this conversation.
No description provided.