-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
additional entry of user domain #18
Comments
As discussed in this Issue: #12 (comment) we already have a defined format for the UserIDs. There are some drawbacks to adding additional fields to the Global Registry Record (GRR): We would just add redundant information. The GRR would be bigger and would have to be checked for inconsistencies, both of which would create a higher load on the Global Registry. Extracting the desired information from the UserID should be much more simple than changing the GRR-related implementations and tests in both Global Registry and Runtime Core. |
we may not change the storage format but provide additional REST interface if needed |
@fbeierle 1.) the url of the domain-registry and (e.g tlabscloud.com) The current format puts both 1) und 2) in one URL and calls it "userID" what is basically wrong because there is much more information in it like domain etc. I think the JSON Formate should be used to clearly state [key,value] pairs. So lets use it. The extra load for the "domain" tag I save in userID-URL because here I need just to write the userID like the tag states it. So don't get me wrong. This is not a big issue. I can deal with it. The only thing is that the naming is missleading and the interface simply not clean if we keep it this way. |
Would a library to do this "splitting" work? No need for everyone to have to write this code over and over. |
@rjflp |
We could either have redundant fields, which we have strong arguments against. Or we change the format of the UserIDs to have several fields instead. I'm unsure, what implications this would have. @pchainho Do you have an opinion on this? |
If I understood it correctly, I would be in favor of having the service domains in the Global Registry (I think I've already proposed something similar somewhere else) On the other hand changing the format of User URLs have a big impact |
@pchainho |
so if i summarize: the api may return
it seems to me that the better choice is what the client services of this api would prefer |
Generally, the "client services" in question here would be mostly the Graph Connector and the Discovery, right? So what would be most beneficial for these two components? One URL or a JSONObject with separated parameters? |
I would say the global registry can be used by any component including Hyperties and should return a json object with addresses compliant with our address model ie the User URL and a second attribute with the user's domain. |
@sgoendoer The main "client service" is the runtime. |
Any solution works for us. We just need to know what to implement. Where else are these URL used? We should use the same data format (json objects) everywhere. |
@jmcrom exactly you name the poblem |
@jmcrom Having a way to request either format as JM proposed will only work with an ugly workaround storing BOTH formats in the dataset. This is because the GReg CAN NOT change the dataset when storing or returning it. Otherwise, the integrity would be voided. How and when are we making a decision to change the current format for the dataset? |
I'd like to see an additional JSON tag "domain" or "service-domain".
In the current datamodel everything is written in "userIDs":
I can work with it somehow and extract the userID and domain from the URL.;
but I think since we use JSON anyway we should clearly name it
"domain":"tlabscloud.com"
"userID": "user://gmail.com/ingo.friese" or "[email protected]"
I think this is not a big issue but makes things much cleaner to implement.
The text was updated successfully, but these errors were encountered: