Make ui/server/registrar and flows targets public#10
Make ui/server/registrar and flows targets public#10jart merged 1 commit intogoogle:masterfrom wolfgangmeyers:flows-visibility
Conversation
|
reviewer = @jart |
|
Can you whitelist the specific targets you need, rather than whitelisting the entire packages? |
There was a problem hiding this comment.
Sorry, this was out of scope. I can change it back if you'd like.
There was a problem hiding this comment.
Please revert this whitespace change.
|
Done! |
There was a problem hiding this comment.
Out of curiosity why do you need to make the ui/server/registrar package visible? The description only mentions making flows visible.
There was a problem hiding this comment.
flows was the first package I encountered that needed visibility changes. Note that the latest commit message mentions flows and registrar packages. I need access to these packages to reuse the login checks (SessionUtils). If you'd prefer to keep these private, I can recreate the logic.
wolfgangmeyers
changed the title
wolfgangmeyers
changed the title
wolfgangmeyers
changed the title
|
Please squash to one commit. |
Needed to reuse server-side XSRF protection
|
done |
Sync from upstream google/nomulus parent
3 participants
This change is necessary in order to directly use the Epp flows
from another project.