Skip to content
This repository was archived by the owner on Jun 22, 2026. It is now read-only.

fix: wildcard should be allowed in alias#3739

Merged
mergify[bot] merged 4 commits into
aws:mainlinefrom
iamhopaul123:alias-fix
Jul 11, 2022
Merged

fix: wildcard should be allowed in alias#3739
mergify[bot] merged 4 commits into
aws:mainlinefrom
iamhopaul123:alias-fix

Conversation

@iamhopaul123

Copy link
Copy Markdown
Contributor

Fixes #3736. https://stackoverflow.com/questions/71170639/error-converting-yaml-to-json-yaml-line-15-did-not-find-expected-alphabetic-o

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the Apache 2.0 License.

@iamhopaul123 iamhopaul123 added the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Jul 8, 2022
@iamhopaul123
iamhopaul123 requested a review from a team as a code owner July 8, 2022 22:01
@iamhopaul123
iamhopaul123 requested review from efekarakus and removed request for a team July 8, 2022 22:01
@iamhopaul123 iamhopaul123 changed the title fix: * should be allowed in alias fix: wildcard should be allowed in alias Jul 8, 2022
- Field: 'host-header'
HostHeaderConfig:
Values: [example.com, foobar.com, v1.foobar.com]
Values: ["example.com", "foobar.com", "v1.foobar.com"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should there be a testcase with "*"?

Comment thread internal/pkg/template/workload.go Outdated
@iamhopaul123 iamhopaul123 removed the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Jul 11, 2022
@mergify
mergify Bot merged commit 4185649 into aws:mainline Jul 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't include "*" in workload alias

4 participants