Skip to content

fix: detect TGT invalidity on 401 AUTH009, not just 404 - #4

Merged
metehanboy merged 1 commit into
mainfrom
fix/tgt-race-parallel-calls
Jul 28, 2026
Merged

metehanboy merged 1 commit into
mainfrom
fix/tgt-race-parallel-calls

Conversation

@metehanboy

Copy link
Copy Markdown
Owner

Şeffaflık servisleri (örn. injection-quantity) TGT geçersizliğini 404 değil 401 + errorCode AUTH009 ile bildiriyor. _is_tgt_invalid sadece 404 kontrol ettiği için bu durum hiç yakalanmıyor, aynı bayat TGT ile 5 kez retry edilip exception fırlatılıyordu.

_is_tgt_invalid artık 401 + body'de AUTH009'u da tanıyor, ve _make_request TGT yenilendiğinde URL'nin yanı sıra headers['TGT']'yi de güncelliyor (bazı servisler TGT'yi header'da gönderiyor, URL'de değil).

Değişiklik Açıklaması

Bu PR'ın ne yaptığını açık ve öz bir şekilde açıklayın.

Değişiklik Türü

Lütfen ilgili kutucuğu işaretleyin:

  • Bug fix (hataları düzelten değişiklik)
  • Yeni özellik (geriye uyumlu değişiklik)
  • Breaking change (geriye uyumlu olmayan değişiklik)
  • Dokümantasyon güncellemesi
  • Kod refactoring
  • Performans iyileştirmesi
  • Test ekleme/güncelleme

İlgili Issue

Bu PR hangi issue'yu çözüyor? (varsa)

Closes #

Değişiklik Detayları

Ne Değişti?

  • Değişiklik 1
  • Değişiklik 2
  • Değişiklik 3

Neden Değişti?

Değişikliğin nedenini açıklayın.

Nasıl Test Edildi?

Değişikliklerin nasıl test edildiğini açıklayın:

  • Mevcut testler geçiyor
  • Yeni testler eklendi
  • Manuel test yapıldı
  • Test sonuçları eklendi

Test Sonuçları

# Test komutları ve sonuçları buraya ekleyin
pytest
# Sonuçlar: ...

Ekran Görüntüleri

Varsa, değişiklikleri gösteren ekran görüntüleri ekleyin.

Kontrol Listesi

PR göndermeden önce aşağıdakileri kontrol edin:

  • Kodumu formatladım (black, isort)
  • Mevcut testler geçiyor
  • Yeni testler ekledim (eğer gerekiyorsa)
  • Dokümantasyonu güncelledim (eğer gerekiyorsa)
  • Commit mesajlarım açıklayıcı
  • Kod standartlarına uyuyorum
  • Type hints ekledim (mümkünse)
  • Docstring'ler ekledim (mümkünse)

Breaking Changes

Bu PR breaking change içeriyor mu?

  • Evet (aşağıda açıklayın)
  • Hayır

Breaking Changes Detayları

Eğer breaking change varsa, detaylarını açıklayın:

  • Ne değişti?
  • Kullanıcılar ne yapmalı?

Ek Notlar

Eklemek istediğiniz başka bir şey var mı?

Review Notları

Reviewer'lar için özel notlar (varsa):

Şeffaflık servisleri (örn. injection-quantity) TGT geçersizliğini
404 değil 401 + errorCode AUTH009 ile bildiriyor. _is_tgt_invalid
sadece 404 kontrol ettiği için bu durum hiç yakalanmıyor, aynı bayat
TGT ile 5 kez retry edilip exception fırlatılıyordu.

_is_tgt_invalid artık 401 + body'de AUTH009'u da tanıyor, ve
_make_request TGT yenilendiğinde URL'nin yanı sıra headers['TGT']'yi
de güncelliyor (bazı servisler TGT'yi header'da gönderiyor, URL'de değil).
@metehanboy
metehanboy merged commit bc238bf into main Jul 28, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant