8000 feat(swabbie): html endpoints for optOut and restore by emjburns · Pull Request #634 · spinnaker/gate · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(swabbie): html endpoints for optOut and restore #634

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

Merged
merged 2 commits into from
Oct 23, 2018

Conversation

emjburns
Copy link
Contributor
@emjburns emjburns commented Oct 16, 2018

Adding an html response for opt out.
Adding an endpoint for getting a resource.
Adding the framework for a restore endpoint, but holding off on exposing it until aws gives us soft delete. If that won't happen, I'll remove the endpoint.


private String getHtmlWithMessage(String namespace, String resourceId, String message) {
return "<body>\n" +
"Swabbie Resource: [namespace=" + namespace + ", resourceId=" + resourceId + "] \n" +
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HTML treats \n as just whitespace. Did you want this to show up on a new line?

Suggested change
"Swabbie Resource: [namespace=" + namespace + ", resourceId=" + resourceId + "] \n" +
"Swabbie Resource: [namespace=" + namespace + ", resourceId=" + resourceId + "] <br>\n" +

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes! Thank you.

@emjburns emjburns merged commit ae315ca into spinnaker:master Oct 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0