Anyway, since the sync also removes old teams when no one is part of that team any more, it seems like the problem of "old teams without an issuer remaining" is not a problem.
Now that you…
It's probably not a real problem. Maybe best to just ignore the problem and let users who are already using this (probably us two) delete the old teams manually with a `delete from teams where…
It looks like the issuer is not set for old teams. I propose two solutions:
- Update the issuer when the team is updated (for the name)
- Delete all openid teams in the migration so…
@konrad thanks, you're absolutely right. The PR should work as is, avoiding duplicate teams being created and leveraging the issuer to compare against existing teams.
I've not found an easy…
@konrad I'll work on that today, I believe it should be reasonable straight forward. One item for discussion though:
Right now, whenever a Team is created, the user who logged in and triggered…
Okay that makes a lot of sense. The first option you mentioned there (make it configurable on a per project level). Imho it should still allow the user to make their choice (among the available…
Okay no worries. Guess it was a nice starter exercise to get into Go and Vue a little bit. I saw the item on the public roadmap a little bit later and figured that it would've been better to first…
Added the scope parameter at the beginning because it was not mentioned anywhere else, before later adding the general schema in openid.md
. Removed them again for all examples as its not needed.
Well I just noticed the oidc id is not enough, it should actually also use the openid provider id as well… since the id is only guaranteed to be unique within that one provider. We should…
I left it in there because it's currently passed to the ErrOIDCTeamDoesNotExist
exception, and then used in various places in the code. If you prefer I can remove those occurrences, although it…
Currently, it's not possible to configure this with environment variables. The documentation of the individual variables is configured from the
config.yml.sample
file and the comments it…
Reproducible for me also in Firefox on Mac OS X, not reproducible in Safari, Chrome or Edge.