Skip to content

Comments

[Inference API] Fix auth exception listener not called bug#139966

Merged
jonathan-buttner merged 3 commits intoelastic:mainfrom
jonathan-buttner:ia-fix-auth-bug
Jan 6, 2026
Merged

[Inference API] Fix auth exception listener not called bug#139966
jonathan-buttner merged 3 commits intoelastic:mainfrom
jonathan-buttner:ia-fix-auth-bug

Conversation

@jonathan-buttner
Copy link
Contributor

This PR fixes a bug where if an exception occurs while making an EIS authorization call the listener may not be called.

@jonathan-buttner jonathan-buttner added >bug auto-backport Automatically create backport pull requests when merged Feature:GenAI Features around GenAI :SearchOrg/Inference Label for the Search Inference team Team:Search - Inference v9.1.10 v9.2.4 v8.19.10 v9.3.1 labels Dec 23, 2025
@elasticsearchmachine
Copy link
Collaborator

Hi @jonathan-buttner, I've created a changelog YAML for you.

@jonathan-buttner jonathan-buttner marked this pull request as ready for review December 23, 2025 20:59
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/search-inference-team (Team:Search - Inference)

@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.1
9.2
8.19
9.3

jonathan-buttner added a commit to jonathan-buttner/elasticsearch that referenced this pull request Jan 6, 2026
…39966)

* Fixing auth catch bug

* Update docs/changelog/139966.yaml
jonathan-buttner added a commit to jonathan-buttner/elasticsearch that referenced this pull request Jan 6, 2026
…39966)

* Fixing auth catch bug

* Update docs/changelog/139966.yaml
elasticsearchmachine pushed a commit that referenced this pull request Jan 6, 2026
…140236)

* Fixing auth catch bug

* Update docs/changelog/139966.yaml
elasticsearchmachine pushed a commit that referenced this pull request Jan 6, 2026
…9966) (#140233)

* [Inference API] Fix auth exception listener not called bug (#139966)

* Fixing auth catch bug

* Update docs/changelog/139966.yaml

* Fixing compile issue

* Revert "Fixing compile issue"

This reverts commit 5e1518c.

* Removing unneeded changes
elasticsearchmachine pushed a commit that referenced this pull request Jan 6, 2026
…39966) (#140235)

* [Inference API] Fix auth exception listener not called bug (#139966)

* Fixing auth catch bug

* Update docs/changelog/139966.yaml

* Fixing compile error
elasticsearchmachine pushed a commit that referenced this pull request Jan 6, 2026
…9966) (#140234)

* [Inference API] Fix auth exception listener not called bug (#139966)

* Fixing auth catch bug

* Update docs/changelog/139966.yaml

* Fixing compile issue

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
szybia added a commit to szybia/elasticsearch that referenced this pull request Jan 7, 2026
* upstream/main: (191 commits)
  Overall Decision for Deciders prioritizes THROTTLE (elastic#140237)
  Apply group by all logic not only to top-level aggregates (elastic#140248)
  [ES|QL] Refactor MV_UNION and MV_INTERSECTION to use shared set operation helper (elastic#139982)
  Avoid reading entire bloom filter file on reader open (elastic#139374)
  Mark bloom filter files for random access (elastic#139375)
  Ensure that the buffer used for ES93BloomFilterStoredFieldsFormat is zeroed (elastic#139034)
  Add busy assertion to avoid race condition for testStalledShardMigrationProperlyDetected (elastic#140230)
  Remove line number check for testTransitiveFindsDeepCallChain (elastic#140228)
  Allow a slight difference in rescored docs (elastic#139931)
  Mute org.elasticsearch.xpack.inference.integration.AuthorizationTaskExecutorIT testCreatesEisChatCompletion_DoesNotRemoveEndpointWhenNoLongerAuthorized elastic#138480
  Start exchange sink fetchers concurrently (elastic#140196)
  Allow allocation to replacement target node on vacate completion (elastic#140150)
  Ignore JNA cleaner threads in SecureHdfsRepositoryAnalysisRestIT (elastic#139925)
  DeterministicQueue refactor and enhancement (elastic#140151)
  Always error out if CCS expression shows up when CCS is not supported (elastic#139009)
  Use IllegalArgumentException over RepositoryException for readonly-repository checks (elastic#140200)
  Guard promql capabilities in AnalyzerTests (elastic#140232)
  [Inference API] Fix flaky AuthorizationTaskExecutorIT tests (elastic#139978)
  Cleaning up exitable vector value impls (elastic#140190)
  [Inference API] Fix auth exception listener not called bug (elastic#139966)
  ...
sidosera pushed a commit to sidosera/elasticsearch that referenced this pull request Jan 7, 2026
…39966)

* Fixing auth catch bug

* Update docs/changelog/139966.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged >bug Feature:GenAI Features around GenAI :SearchOrg/Inference Label for the Search Inference team Team:Search - Inference v8.19.10 v9.1.10 v9.2.4 v9.3.1 v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants