Skip to content

Conversation

@tvernum
Copy link
Contributor

@tvernum tvernum commented Aug 19, 2025

This commit limits the maximum depth of a filter expression such as abc.*.d*.*e.f.*

These expressions are most commonly used in filter_path on in a URL, and the limit has been set to 500 - in effect such a filter may only contain 500 . separators between field expressions.

This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Aug 19, 2025
Copy link
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

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

LGTM

@tvernum tvernum added :Core/Infra/REST API REST infrastructure and utilities and removed needs:triage Requires assignment of a team area label labels Aug 19, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Aug 19, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine
Copy link
Collaborator

Hi @tvernum, I've created a changelog YAML for you.

@tvernum tvernum enabled auto-merge (squash) August 19, 2025 07:11
@tvernum tvernum merged commit c904039 into elastic:main Aug 20, 2025
34 checks passed
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Aug 20, 2025
This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.

Backport of: elastic#133113
elasticsearchmachine pushed a commit that referenced this pull request Aug 20, 2025
This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.

Backport of: #133113
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Aug 20, 2025
This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.

Backport of: elastic#133113
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Aug 20, 2025
This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.

Backport of: elastic#133113
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Aug 20, 2025
This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.

Backport of: elastic#133113
elasticsearchmachine pushed a commit that referenced this pull request Aug 20, 2025
This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.

Backport of: #133113
elasticsearchmachine pushed a commit that referenced this pull request Aug 20, 2025
This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.

Backport of: #133113
elasticsearchmachine pushed a commit that referenced this pull request Aug 20, 2025
This commit limits the maximum depth of a filter expression such as
`abc.*.d*.*e.f.*`

These expressions are most commonly used in `filter_path` on in a URL,
and the limit has been set to 500 - in effect such a filter may only
contain 500 `.` separators between field expressions.

Backport of: #133113
szybia added a commit to szybia/elasticsearch that referenced this pull request Aug 20, 2025
…improv

* upstream/main: (58 commits)
  Fixing flaky LoggedExec (tests) (elastic#133215)
  CPS search should not use `skip_unavailable` (elastic#132927)
  Don't fail search if bottom doc can't be formatted (elastic#133188)
  Mute org.elasticsearch.xpack.esql.action.RandomizedTimeSeriesIT testGroupByNothing elastic#133225
  Mute org.elasticsearch.index.codec.tsdb.es819.ES819TSDBDocValuesFormatTests testSortedSetDocValuesWithSkipperSmall elastic#133224
  Mute org.elasticsearch.index.codec.tsdb.es819.ES819TSDBDocValuesFormatTests testSortedNumberMergeAwayAllValuesWithSkipper elastic#133223
  Adding support for index.number_of_replicas to data stream settings (elastic#132748)
  Mute org.elasticsearch.index.codec.tsdb.es819.ES819TSDBDocValuesFormatTests testSortedDocValuesSingleUniqueValue elastic#133221
  Fix VectorSimilarityFunctionsIT (elastic#133206)
  Mute org.elasticsearch.xpack.esql.action.RandomizedTimeSeriesIT testGroupBySubset elastic#133220
  Increase the number of FORK branches in ForkGenerator (elastic#132019)
  Mute org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests testBlockLoader {preference=Params[syntheticSource=true, preference=STORED]} elastic#133218
  Mute org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests testBlockLoader {preference=Params[syntheticSource=true, preference=DOC_VALUES]} elastic#133217
  Mute org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests testBlockLoader {preference=Params[syntheticSource=true, preference=NONE]} elastic#133216
  Set default processor allocation for test clusters (elastic#133204)
  Add mapper for exponential histograms (elastic#132493)
  Fix offset handling in Murmur3Hasher (elastic#133193)
  unmute testDoesNotResolveClosedIndex (elastic#133115)
  Fix an AWS SDK v2 release note (elastic#133155)
  Limit the depth of a filter (elastic#133113)
  ...
szybia added a commit to szybia/elasticsearch that referenced this pull request Aug 20, 2025
* upstream/main: (58 commits)
  Fixing flaky LoggedExec (tests) (elastic#133215)
  CPS search should not use `skip_unavailable` (elastic#132927)
  Don't fail search if bottom doc can't be formatted (elastic#133188)
  Mute org.elasticsearch.xpack.esql.action.RandomizedTimeSeriesIT testGroupByNothing elastic#133225
  Mute org.elasticsearch.index.codec.tsdb.es819.ES819TSDBDocValuesFormatTests testSortedSetDocValuesWithSkipperSmall elastic#133224
  Mute org.elasticsearch.index.codec.tsdb.es819.ES819TSDBDocValuesFormatTests testSortedNumberMergeAwayAllValuesWithSkipper elastic#133223
  Adding support for index.number_of_replicas to data stream settings (elastic#132748)
  Mute org.elasticsearch.index.codec.tsdb.es819.ES819TSDBDocValuesFormatTests testSortedDocValuesSingleUniqueValue elastic#133221
  Fix VectorSimilarityFunctionsIT (elastic#133206)
  Mute org.elasticsearch.xpack.esql.action.RandomizedTimeSeriesIT testGroupBySubset elastic#133220
  Increase the number of FORK branches in ForkGenerator (elastic#132019)
  Mute org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests testBlockLoader {preference=Params[syntheticSource=true, preference=STORED]} elastic#133218
  Mute org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests testBlockLoader {preference=Params[syntheticSource=true, preference=DOC_VALUES]} elastic#133217
  Mute org.elasticsearch.index.mapper.blockloader.IpFieldBlockLoaderTests testBlockLoader {preference=Params[syntheticSource=true, preference=NONE]} elastic#133216
  Set default processor allocation for test clusters (elastic#133204)
  Add mapper for exponential histograms (elastic#132493)
  Fix offset handling in Murmur3Hasher (elastic#133193)
  unmute testDoesNotResolveClosedIndex (elastic#133115)
  Fix an AWS SDK v2 release note (elastic#133155)
  Limit the depth of a filter (elastic#133113)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants