reva
reva copied to clipboard
UserId idp may have prefix
The idp field of a UserId should identify the identity provider of the user, but it's unclear from the current documentation of cs3apis whether this should be done with a domain (e.g. 'cernbox.cern.ch') or with an origin (e.g. 'https://cernbox.cern.ch').
Recently, the users.demo.json file in the ocmd example switched from the first to the second format.
In https://github.com/cs3org/cs3apis/pull/159 I proposed to make the second format the prescribed one, but we don't want to make this a breaking change, so reva should start supporting origin-format for userid idp in addition to supporting domain-format as it does currently.
We should to make the provider authorizer interceptor accept both formats, I'll create a PR.
@michielbdejong We want to finalize this (in the context of https://github.com/cs3org/reva/pull/3121). Can you confirm that the proposed format of the idp field, as origin (eg. https://cernbox.cern.ch) is definite ?
In https://github.com/cs3org/cs3apis/pull/159#discussion_r766662560 @wkloucek proposed a 3rd option, namely a full URL. So the options are:
- domain, e.g. "cernbox.cern.ch"
- origin, e.g. "https://cernbox.cern.ch"
- full URL, e.g. "https://cernbox.cern.ch/some/path"
I'm OK with whatever you prefer!