Skip to content

Deprecated APIs removal - #9818

Merged
zwoop merged 3 commits into
apache:masterfrom
zwoop:DeprecatedAPIs
Jun 13, 2023
Merged

Deprecated APIs removal#9818
zwoop merged 3 commits into
apache:masterfrom
zwoop:DeprecatedAPIs

Conversation

@zwoop

@zwoop zwoop commented Jun 9, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@zwoop zwoop added this to the 10.0.0 milestone Jun 9, 2023
traeak
traeak previously approved these changes Jun 9, 2023

@traeak traeak left a comment

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.

These look fine to me.

@shukitchan shukitchan left a comment

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.

@zwoop

zwoop commented Jun 12, 2023

Copy link
Copy Markdown
Contributor Author

Please update the documents for lua plugin here as well.

https://docs.trafficserver.apache.org/en/latest/admin-guide/plugins/lua.en.html#ts-http-client-packet-tos-set
https://docs.trafficserver.apache.org/en/latest/admin-guide/plugins/lua.en.html#ts-http-server-packet-tos-set

@shukitchan Removed from the docs (but in the .rst file). The naming here made these difficult to find :).

@zwoop
zwoop merged commit 9a11319 into apache:master Jun 13, 2023
@zwoop
zwoop deleted the DeprecatedAPIs branch June 13, 2023 21:43
bryancall pushed a commit to bryancall/trafficserver that referenced this pull request Jun 14, 2023
* Removes deprecated APIs

* Removes the TOS APIs

* Removes the TOS from lua docs
@bryancall bryancall mentioned this pull request Aug 14, 2024
91 tasks
maskit added a commit to maskit/trafficserver that referenced this pull request Aug 15, 2024
maskit added a commit that referenced this pull request Aug 15, 2024
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