Skip to content
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

IDM: remove Domain Registry query for Data Object Reporter #145

Open
pchainho opened this issue Dec 7, 2016 · 6 comments
Open

IDM: remove Domain Registry query for Data Object Reporter #145

pchainho opened this issue Dec 7, 2016 · 6 comments
Assignees

Comments

@pchainho
Copy link
Contributor

pchainho commented Dec 7, 2016

In the scope of Mutual Authentication process, it seems the Domain Registry is queried to retrieve the Reporter of a Data Object. This query is not required since the Reporter URL is provided in the Data Object itself ie it can be found at message.body.value.reporter.

It should be noted that this Domain Registry reporter query conflicts with the Query per Data Object URL needed by P2P Stubs.

@tiagolb
Copy link
Contributor

tiagolb commented Jan 26, 2017

@pchainho the message you are referring to is the one used in the RuntimeCoreCtx.js file (function doMutualAuthentication), which in turn calls the doMutualAuthentication function in IDM, where the registry is queried incorrectly?

RuntimeCoreCtx.js:
demutualruntimecorectx

IdentityModule.js:
domutualidm

@tiagolb
Copy link
Contributor

tiagolb commented Feb 6, 2017

There's a discrepancy regarding the implementation and the specs, which means someone needs to add the reporter field to the message. So far no further development has been made.

@tiagolb
Copy link
Contributor

tiagolb commented Feb 13, 2017

@pchainho Who is responsible for editing the reporter field in this message? Without the reporter field in the incoming message, the IDM doesn't have access to it and as such I cannot remove the Domain Registry query.

@tiagolb
Copy link
Contributor

tiagolb commented Feb 16, 2017

@pchainho any news about this issue?

1 similar comment
@tiagolb
Copy link
Contributor

tiagolb commented Feb 24, 2017

@pchainho any news about this issue?

@pchainho
Copy link
Contributor Author

pchainho commented Jul 4, 2017

The implementation was aligned with the specs and the info about the data object reporter should now be available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants