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(cors): Support private network access #11523

Merged
merged 3 commits into from
Sep 11, 2023

Conversation

liverpool8056
Copy link
Contributor

Summary

The Access-Control-Request-Private-Network header in crossing-origin pre-light requests is supported through adding a new parameter private_network which determines whether the Access-Control-Allow-Private-Network header should be carried in the response headers with true as the value.

Checklist

Full changelog

  • [Implement ...]

Issue reference

FTI-5258

crossing-origin prelight request. A new parameter `private_network` is
added to determined whether the `Access-Control-Allow-Private-Network`
header should be carried in the response headers with `true` as the value.
@liverpool8056 liverpool8056 force-pushed the feat/cors-private-network-support branch from a78e7fe to c9ae4cb Compare September 8, 2023 04:17
@liverpool8056 liverpool8056 requested a review from ms2008 September 8, 2023 04:58
Copy link
Contributor

@ms2008 ms2008 left a comment

Choose a reason for hiding this comment

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

New fields need to be added to the removed fields to support mixed-version clusters. It is necessary to raise a PR to https://github.com/Kong/docs.konghq.com to introduce this feature as well.

@pull-request-size pull-request-size bot added size/L and removed size/M labels Sep 8, 2023
@ms2008 ms2008 changed the title feat(cors): Support in prelight request feat(cors): Support private network access Sep 8, 2023
@liverpool8056 liverpool8056 requested a review from ms2008 September 8, 2023 08:11
@windmgc windmgc merged commit c738e45 into master Sep 11, 2023
@windmgc windmgc deleted the feat/cors-private-network-support branch September 11, 2023 13:14
@@ -35,11 +35,11 @@ end
local function get_plugin(node_id, node_version, name)
local res, err = cluster_client({ id = node_id, version = node_version })
assert.is_nil(err)
assert.is_table(res and res.config and res.config.plugins,
assert.is_table(res and res.config_table and res.config_table.plugins,
Copy link
Contributor

Choose a reason for hiding this comment

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

liverpool8056 added a commit that referenced this pull request Feb 22, 2024
The Access-Control-Request-Private-Network header in crossing-origin pre-light requests is supported through adding a new parameter private_network which determines whether the Access-Control-Allow-Private-Network header should be carried in the response headers with true as the value.

FTI-5258
ms2008 pushed a commit that referenced this pull request Feb 26, 2024
…12599)

* feat(cors): Support private network access (#11523)

The Access-Control-Request-Private-Network header in crossing-origin pre-light requests is supported through adding a new parameter private_network which determines whether the Access-Control-Allow-Private-Network header should be carried in the response headers with true as the value.

* change version number in removed_fileds and spce for compatibility

FTI-5258
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.

4 participants