Skip to content

Update http.md#31347

Merged
tavplubix merged 2 commits intoClickHouse:masterfrom
lavrinenkoe:patch-2
Nov 12, 2021
Merged

Update http.md#31347
tavplubix merged 2 commits intoClickHouse:masterfrom
lavrinenkoe:patch-2

Conversation

@lavrinenkoe
Copy link
Copy Markdown
Contributor

@lavrinenkoe lavrinenkoe commented Nov 12, 2021

tag throws error in config. It seems should be there

Changelog category (leave one):

  • Documentation (changelog entry is not required)

Detailed description / Documentation draft:

else if (filter_type == "methods")
addFilter(methodsFilter(config, prefix + ".methods"));

<method> tag throws error in config. It seems <methods> should be there
@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Nov 12, 2021
@tavplubix tavplubix mentioned this pull request Nov 12, 2021
@tavplubix tavplubix self-assigned this Nov 12, 2021
@robot-clickhouse robot-clickhouse added pr-documentation Documentation PRs for the specific code PR and removed pr-bugfix Pull request with bugfix, not backported by default labels Nov 12, 2021
Copy link
Copy Markdown
Member

@tavplubix tavplubix left a comment

Choose a reason for hiding this comment

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

Nice catch, thanks!

@tavplubix tavplubix merged commit e8f745d into ClickHouse:master Nov 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-documentation Documentation PRs for the specific code PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants