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(plugins): adjust content-type explanation for new charset option #5325

Closed
wants to merge 51 commits into from

Conversation

tysoekong
Copy link
Contributor

Description

Added config schema explanation, as per PR Kong/kong#10533

The PR adds just one config option to the restricted list available for http-log's "content_type" schema entry.

Target is for it to end up at this page, upon the next Kong minor release: https://docs.konghq.com/hub/kong-inc/http-log/

Testing instructions

Netlify link:

Checklist

  • Review label added
  • PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

@tysoekong tysoekong added the review:general Review for general accuracy and presentation. Does the doc work? Does it output correctly? label Mar 21, 2023
@tysoekong tysoekong requested a review from a team as a code owner March 21, 2023 14:34
@netlify
Copy link

netlify bot commented Mar 21, 2023

Deploy Preview for kongdocs ready!

Name Link
🔨 Latest commit b0a71cf
🔍 Latest deploy log https://app.netlify.com/sites/kongdocs/deploys/6419c11ac1f63f000865b9fc
😎 Deploy Preview https://deploy-preview-5325--kongdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@tysoekong tysoekong changed the base branch from main to release/gateway-3.3 March 21, 2023 14:34
Leopoldthecoder and others added 25 commits March 21, 2023 14:36
version the description for the RLA plugin
So that it pulls the latest changes from master and creates a PR against
main.
* Update Kuma submodule

* [mesh] Add community section to navigation bar (#5275)

update mesh nav bar

---------

Co-authored-by: Angel <[email protected]>
* chore(*): docker read-only mode docs

* Apply suggestions from code review

---------

Co-authored-by: Diana <[email protected]>
fix kic manifest links
Done using the following regex in VSCode. It avoids images using the
leading [^!]

([^!]\[.+\]\(\/[^#\s]+[^/])\)
Replaced with: $1/)
* Remove datadog-tracing plugin

* remove plugin

* remove from version errors
* remove all macos homebrew instructions and mentions; rewrite macos topics after 3.0 to link to Docker

* redirect and typo

* appease vale
nephel and others added 26 commits March 21, 2023 14:36
* Update index.md

* format note, fix link, grammar

---------

Co-authored-by: lena-larionova <[email protected]>
* split plugin instructions into kong vs third-party

* code block best practices; split notes into their own topic; move screenshot info

* draft of user interfaces

* slight adjustments to templates

* improvements to the plugin doc + user interfaces

* Apply suggestions from code review

Co-authored-by: Angel <[email protected]>

* use consistent formatting; fix typo; add supporting links

* fix links

* vale

---------

Co-authored-by: Angel <[email protected]>
* 3.2.2.0 Changelog

https://konghq.atlassian.net/browse/DOCU-3047

* Update changelog.md

* Update changelog.md

* Update changelog.md

* Apply suggestions from code review

Co-authored-by: Diana <[email protected]>

* Update changelog.md

adding known issue from slack

* Update changelog.md

* Update changelog.md

* Apply suggestions from code review

Co-authored-by: lena-larionova <[email protected]>

---------

Co-authored-by: Diana <[email protected]>
Co-authored-by: lena-larionova <[email protected]>
Bumps [activesupport](https://github.com/rails/rails) from 7.0.4.1 to 7.0.4.3.
- [Release notes](https://github.com/rails/rails/releases)
- [Changelog](https://github.com/rails/rails/blob/v7.0.4.3/activesupport/CHANGELOG.md)
- [Commits](rails/rails@v7.0.4.1...v7.0.4.3)

---
updated-dependencies:
- dependency-name: activesupport
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
* Added aws-request-signing.

* Rename _index.yaml to _index.md

correcting file extension

* Delete versions.yml

Removing version file as only required for Kong plug-ins

* Update _index.md

removing instances of "  minimum_version: 1.0.0", not required.

* Fixed config.

* Better examples.

* Removed extra 'config:'

* Update auto-ignore.txt

adding new variables to linting

* Apply suggestions from code review

Co-authored-by: lena-larionova <[email protected]>

* fix bad indent

---------

Co-authored-by: Amy Goldsmith <[email protected]>
Co-authored-by: lena-larionova <[email protected]>
* Add system accounts UI description.

* Update app/konnect/org-management/system-accounts.md

* Update app/konnect/org-management/system-accounts.md

---------

Co-authored-by: Angel <[email protected]>
* space out navbar to fit search; minor adjustments for matching konghq style

* center vertically

* fix colour flashing
mark consumer groups as supported in 1.17.x and later
* default rl strategy in konnect is now local

* fix links and remove outdated text

* accurate limitations for rate limiting strategies

* revert one of the links

* fix typo
* add enterprise plugin priority changes to 3.0 changelog

* fix broken links
add passphrase parameter to certificates object
@tysoekong tysoekong force-pushed the feat/http-log-schema-update branch from 909d5bd to b0a71cf Compare March 21, 2023 14:37
@tysoekong tysoekong closed this Mar 21, 2023
@tysoekong tysoekong deleted the feat/http-log-schema-update branch March 21, 2023 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:check-skipped:generated-files review:general Review for general accuracy and presentation. Does the doc work? Does it output correctly?
Projects
None yet
Development

Successfully merging this pull request may close these issues.