skip org creation of org is empty

georchestra / georchestra-gateway

GNU General Public License v3.0

0 stars 6 forks source link

skip org creation of org is empty #78

Closed pmauduit closed 10 months ago

pmauduit commented 10 months ago

some IdP (e.g. FranceConnect) seem to return a blank ("") organization, we cannot rely on the try {} catch (NameNotFoundException) here, but at least we can just skip the org creation if we don't have at least a name to create it.

Tests: tried with the IT testsuite from the branch still in review from PR #63, but which does not exist yet ... A simple case where preauth-org: '' is sufficient to reproduce the issue.

fvanderbiest commented 10 months ago

:+1:

pmauduit commented 10 months ago

There is also this one, but at this point I can't figure out who is right to be honest ... https://github.com/georchestra/georchestra-gateway/pull/79

pmauduit commented 10 months ago

superseded by #79 :man_shrugging: