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

Revert "fix(appname): encodeURIComponent for app name (#8586)" #8627

Merged
merged 1 commit into from
Oct 6, 2020

Conversation

caseyhebebrand
Copy link
Contributor

Reverting the PR.

@spinnakerbot
Copy link
Contributor

The following commits need their title changed:

Please format your commit title into the form:

<type>(<scope>): <subject>, e.g. fix(kubernetes): address NPE in status check

This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here.

@caseyhebebrand
Copy link
Contributor Author

@german-muzquiz Please take a look. We need to rollout some changes on top of this PR, so we thought the easiest thing would be to revert until the issue could be resolved .The issue was that all of the slashes were replaced with space for any endpoint calls.

@german-muzquiz
Copy link
Contributor

@caseyhebebrand I'm trying to uderstand the issue, are you saying that after merging this PR, any and all API calls replace slashes by spaces? I didn't see that in my tests, can you elaborate a bit more for exactly which API calls did you see this?

@christopherthielen
Copy link
Contributor

@german-muzquiz This lives in proprietary netflix code (think: plugins) that uses API.one('with/slashes'). I'll add the linter rule first to enable folks to update their code, then we can un-revert this PR.

@german-muzquiz german-muzquiz added the ready to merge Reviewed and ready for merge label Oct 6, 2020
@mergify mergify bot merged commit 885cd16 into spinnaker:master Oct 6, 2020
christopherthielen added a commit that referenced this pull request Oct 6, 2020
885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
christopherthielen added a commit that referenced this pull request Oct 6, 2020
885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
christopherthielen added a commit that referenced this pull request Oct 6, 2020
885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
christopherthielen added a commit that referenced this pull request Oct 6, 2020
885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
christopherthielen added a commit that referenced this pull request Oct 6, 2020
885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
christopherthielen added a commit that referenced this pull request Oct 6, 2020
885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
christopherthielen added a commit that referenced this pull request Oct 6, 2020
885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
christopherthielen added a commit that referenced this pull request Oct 6, 2020
885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
mergify bot pushed a commit that referenced this pull request Oct 6, 2020
…e@0.0.517 docker@0.0.61 google@0.0.22 oracle@0.0.10 tencentcloud@0.0.7 (#8630)

* chore(amazon): publish amazon@0.0.270

885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)

* chore(azure): publish azure@0.0.256

885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)

* chore(cloudfoundry): publish cloudfoundry@0.0.102

885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)

* chore(core): publish core@0.0.517

885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)

* chore(docker): publish docker@0.0.61

885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)

* chore(google): publish google@0.0.22

885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)

* chore(oracle): publish oracle@0.0.10

885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)

* chore(tencentcloud): publish tencentcloud@0.0.7

885cd16 Revert "fix(appname): encodeURIComponent for app name (#8586)" (#8627)
@caseyhebebrand caseyhebebrand deleted the revert-encoding branch February 24, 2021 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Reviewed and ready for merge target-release/1.23
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants