misc: skip tenants with unallowed hostname (#4794) #1244

fpeters · 2024-03-10T16:58:13+01:00

fpeters commented

2024-03-10 16:58:13 +01:00

Un petit peu différent du plan proposé dans le ticket puisqu'on peut utiliser le fichier site-options.cfg plutôt qu'inventer un nouveau fichier; voici donc, en utilisant un nom d'option (allowed_hostname) dans l'esprit similaire au ALLOW_HOSTS de django.

fpeters added 1 commit 2024-03-10 16:58:14 +01:00

gitea/wcs/pipeline/head There was a failure building this commit Details

50dc64df36 misc: skip tenants with unallowed hostname (#4794 )

fpeters reviewed 2024-03-10 16:59:03 +01:00

wcs/qommon/publisher.py Outdated

						
				@ -1012,0 +1019,4 @@

				                    if tenant != allowed_hostname:

				                        continue

				                except (configparser.NoOptionError, configparser.NoSectionError):

				                    pass  # legacy

fpeters commented

2024-03-10 16:59:02 +01:00

À noter que la modification à hobo_deploy va faire en sorte que les sites existants auront leur site-options.cfg complété (lors de l'appel à hobo_deploy --redeploy), donc on pourrait imaginer plus tard retirer cette partie "legacy".

À noter que la modification à hobo_deploy va faire en sorte que les sites existants auront leur site-options.cfg complété (lors de l'appel à `hobo_deploy --redeploy`), donc on pourrait imaginer plus tard retirer cette partie "legacy".

fpeters force-pushed wip/4794-mark-vhost-name from 50dc64df36 to e656a4d1b9

2024-03-10 17:40:11 +01:00

Compare

fpeters changed title from ~~WIP: misc: skip tenants with unallowed hostname (#4794)~~ to misc: skip tenants with unallowed hostname (#4794)

2024-03-10 17:58:54 +01:00

tnoel commented

2024-03-11 10:04:20 +01:00

Un petit peu différent du plan proposé dans le ticket puisqu'on peut utiliser le fichier site-options.cfg plutôt qu'inventer un nouveau fichier; voici donc, en utilisant un nom d'option (allowed_hostname) dans l'esprit similaire au ALLOW_HOSTS de django.

Le soucis que je vois c'est que ça va créer une différence avec les autres logiciels qui ont des cron ; mais on peut dire aussi qu'on s'en fiche et qu'on verra à gérer ça autrement dans hobo.multitenant... (sauf que je vois pas trop d'autre technique que gérer un nouveau fichier)

> Un petit peu différent du plan proposé dans le ticket puisqu'on peut utiliser le fichier site-options.cfg plutôt qu'inventer un nouveau fichier; voici donc, en utilisant un nom d'option (allowed_hostname) dans l'esprit similaire au ALLOW_HOSTS de django. Le soucis que je vois c'est que ça va créer une différence avec les autres logiciels qui ont des cron ; mais on peut dire aussi qu'on s'en fiche et qu'on verra à gérer ça autrement dans hobo.multitenant... (sauf que je vois pas trop d'autre technique que gérer un nouveau fichier)

fpeters commented

2024-03-11 13:34:50 +01:00

Le soucis que je vois c'est que ça va créer une différence avec les autres logiciels qui ont des cron ; mais on peut dire aussi qu'on s'en fiche et qu'on verra à gérer ça autrement dans hobo.multitenant... (sauf que je vois pas trop d'autre technique que gérer un nouveau fichier)

Si quelque chose s'adopte dans hobo on pourra adapter wcs, en attendant il n'y a même pas de ticket pour se prémunir de ça côté hobo, et on a finalement vécu avec pendant très longtemps sans soucis.

> Le soucis que je vois c'est que ça va créer une différence avec les autres logiciels qui ont des cron ; mais on peut dire aussi qu'on s'en fiche et qu'on verra à gérer ça autrement dans hobo.multitenant... (sauf que je vois pas trop d'autre technique que gérer un nouveau fichier) Si quelque chose s'adopte dans hobo on pourra adapter wcs, en attendant il n'y a même pas de ticket pour se prémunir de ça côté hobo, et on a finalement vécu avec pendant très longtemps sans soucis.

gitea/wcs/pipeline/head This commit looks good

Details

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1244