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

feat: Implement a wildcard in http allowed_domains to allow all domains #422

Merged
merged 18 commits into from
May 11, 2022

Conversation

Mossaka
Copy link
Contributor

@Mossaka Mossaka commented May 2, 2022

This PR fixes the issue #421 .

It does

  1. add a wildcard "*" option to allowed_http_hosts in HTTP component.
  2. add an example to demonstrate how to use it to make outbound http call in rust

Note: this PR has a dependency on deislabs/wasi-experimental-http#91. Once the upstream PR is merged, this PR will go off-draft.

crates/outbound-http/Cargo.toml Outdated Show resolved Hide resolved
crates/outbound-http/src/lib.rs Outdated Show resolved Hide resolved
Copy link
Collaborator

@lann lann left a comment

Choose a reason for hiding this comment

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

Discuss security: #421 (comment)

@Mossaka Mossaka marked this pull request as ready for review May 7, 2022 03:16
@Mossaka Mossaka requested a review from lann May 10, 2022 05:30
crates/outbound-http/Cargo.toml Outdated Show resolved Hide resolved
crates/outbound-http/src/lib.rs Outdated Show resolved Hide resolved
@Mossaka Mossaka requested a review from lann May 11, 2022 05:12
@lann lann merged commit 09ad9d7 into fermyon:main May 11, 2022
danbugs pushed a commit to danbugs/spin that referenced this pull request May 12, 2022
…ns (fermyon#422)

* added wildcard host and a rust outbound http example
* updated outbound http upstream

Signed-off-by: Jiaxiao Zhou <[email protected]>
Signed-off-by: danbugs <[email protected]>
danbugs pushed a commit to danbugs/spin that referenced this pull request May 12, 2022
…ns (fermyon#422)

* added wildcard host and a rust outbound http example
* updated outbound http upstream

Signed-off-by: Jiaxiao Zhou <[email protected]>
danbugs pushed a commit to danbugs/spin that referenced this pull request May 12, 2022
…ns (fermyon#422)

* added wildcard host and a rust outbound http example
* updated outbound http upstream

Signed-off-by: Jiaxiao Zhou <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants