Skip to content

chore: remove stream_proxy.only in config-default.yaml#10337

Merged
moonming merged 1 commit into
apache:masterfrom
juststillthinking:remove-unnecessary-config
Oct 17, 2023
Merged

chore: remove stream_proxy.only in config-default.yaml#10337
moonming merged 1 commit into
apache:masterfrom
juststillthinking:remove-unnecessary-config

Conversation

@juststillthinking

Copy link
Copy Markdown
Contributor

Description

Remove the configuration item apisix.stream_proxy.only, the L4/L7 proxy needs to be enabled through the configuration item apisix.proxy_mode: #9607

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@moonming

Copy link
Copy Markdown
Member

@kayx23 do we have docs for this feature?

@moonming
moonming merged commit 33ee8d6 into apache:master Oct 17, 2023
@kayx23

kayx23 commented Oct 17, 2023

Copy link
Copy Markdown
Member

@moonming APISIX docs yes (stream proxy]; API7 docs not yet.

@moonming

Copy link
Copy Markdown
Member

@moonming APISIX docs yes (stream proxy]; API7 docs not yet.

This document is too simple and the proposal is better than it #9600
We can add to todo list.

@kayx23

kayx23 commented Oct 17, 2023

Copy link
Copy Markdown
Member

Sounds good - added.

hongbinhsu pushed a commit to fitphp/apix that referenced this pull request Nov 1, 2023
* upstream/master: (83 commits)
  fix: make install failed on mac (apache#10403)
  feat(zipkin): add variable (apache#10361)
  test(clickhouse-logger): to show that different endpoints will be chosen randomly (apache#8777)
  chore(deps): bump actions/setup-node from 3.8.1 to 4.0.0 (apache#10381)
  ci: fix the grpc test error (apache#10388)
  ci: trigger ci when doc-lint.yml changes (apache#10382)
  docs: fix usage of incorrect default admin api port (apache#10391)
  feat: Add authorization params to openid-connect plugin (apache#10058)
  feat: integrate authz-keycloak with secrets resource (apache#10353)
  fix(traffic-split): post_arg match fails because content-type contains charset (apache#10372)
  fix(consul): worker will not exit while reload or quit (apache#10342)
  chore: update rules for unresponded issues (apache#10354)
  docs: Update APISIX usecases in README (apache#10358)
  test: use http2 to test limit-req plugin (apache#10334)
  test: use http2 to test limit-conn plugin (apache#10332)
  chore: remove stream_proxy.only in config-default.yaml (apache#10337)
  docs: update underscore to hyphen in HTTP headers in `response-rewrite` plugin (apache#10347)
  fix: typos in comments (apache#10330)
  feat: support config stream_route upstream in service (apache#10298)
  fix: keep healthcheck target state when upstream changes (apache#10312)
  ...
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.

3 participants