-
Notifications
You must be signed in to change notification settings - Fork 184
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
Omit some lookups in gateway #9715
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Protest with naming. Code looks good though 👍
@kobergj mea culpa |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome 👍
ad97158
to
10764b4
Compare
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> extract method Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
10764b4
to
297cc77
Compare
Quality Gate passedIssues Measures |
Omit some lookups in gateway
The CS3 gateway can directly to itself when it wants to talk to the registries running in the same reva runtime.
Requires a reva bump that is part of this PR to pull in cs3org/reva#4744
part of #8589