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.

Sign up for GitHub

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

Jump to bottom

moving the tokentype field from the body section to the header section of the jwt #69

Merged
merged 6 commits into from
Jun 7, 2023
Merged

moving the tokentype field from the body section to the header section of the jwt #69

merged 6 commits into from
Jun 7, 2023

Conversation

ilkerCelimli
Copy link
Contributor

TokenType değeri claims'den alınıp token headerine taşındı,

yeni token Access

eyJ0eXAiOiJCZWFyZXIiLCJhbGciOiJSUzUxMiJ9.eyJqdGkiOiI3OWIzMTZiZC01YzdiLTQ1NTctODNlOS1hOTZmYzNiNDMzMjkiLCJpc3MiOiJBWVMiLCJpYXQiOjE2ODU4NzE0ODcsImV4cCI6MTY4NTg3ODY4Nywib3JnYW5pemF0aW9uSWQiOiI3N2VjZTI1Ni1iZjBlLTRiYmUtODAxZC0xNzMwODNmOGJkY2YiLCJ1c2VyTGFzdE5hbWUiOiJTaXN0ZW1pIiwidXNlclR5cGUiOiJBRE1JTiIsInVzZXJGaXJzdE5hbWUiOiJBZmV0IFnDtm5ldGltIiwidXNlcm5hbWUiOiJheXMtYWRtaW4ifQ.N4Yxu7bA49Qx7l0YH6tfUfdL90kI8IJcYGAVA2FUvohvKywCIOne9fQuLpNZRyWJnr4mqEp9VZIdjslGVdsFi0ncR2_rB6set2C86xFU_rPtHa5xcfwuj_LmByJoMFKi-agRgzQK4JjeZXmL4zDLERkRi7ZE8NCAZhLiEtXRSHhobBzMQGVkQkV1mPPWhGgYx6QTfRnw5kjT1H9DR4yNxXqpAgiG2qQhrHioQLjMrCaD7FDplOW3KnV3fQ2FzMRdY7QWp5575wiK23Tt-VBbherA_at5D01gzTDkng8YbfKiqcRhm4S8W-fV5RA6ho2p-vqkwlYSD4hf57QmggznBg

Refresh
eyJ0eXAiOiJCZWFyZXIiLCJhbGciOiJSUzUxMiJ9.eyJqdGkiOiJkYTI3MDBkMi0zM2ExLTQ3NGEtYmZjYS05MjhjNDc1Y2NkMDkiLCJpc3MiOiJBWVMiLCJpYXQiOjE2ODU4NzE0ODcsImV4cCI6MTY4NTk1Nzg4NywidXNlcm5hbWUiOiJheXMtYWRtaW4ifQ.mKR_UMb6kQHENfZmr8F50UhtW6KQ-TKLkm21ve0dFqqKlZk_8DUupiZmb8ABzyR8EIVRRjMEQVMglyRNtK-vVfgFQCTETWFjqc6Fw3N-ViefgTVM_rLwcu26hHTvmh7MQdS6X34csYTk9eaAH0FJPqbxhGmaAoFCZwyX1s_WWHUZy-iUbLIQUgkqM3MNt8VImLq8ORwXduyL1WU0FywhX1mQmCF-HhEapO-2RTsRT9otCeAZUQzITfHQpRJ45Q6ONNJqn2Volav9vKdttLlFq288KNYR9L2iz9U331rFfVnFxvFwxr_NCq65z1_WZxs7yMmWOarKOKynvX24fbI2eg

jwt.io'dan kontrol edilebilir.

@ilkerCelimli ilkerCelimli requested a review from agitrubard as a code owner June 4, 2023 09:39
@ilkerCelimli ilkerCelimli linked an issue Jun 4, 2023 that may be closed by this pull request
Moving The tokenType Field From The Body Section To The Header Section Of The JWT #60
Closed
@agitrubard
Copy link
Collaborator

@ilkerCelimli Pull request başlığına Feature/ gibi bir prefix eklememize ihtiyacımız bulunmuyor, böyle bir standardımız da yok. Kaldırabilir miyiz?

agitrubard
agitrubard requested changes Jun 4, 2023
View reviewed changes
Copy link
Collaborator

@agitrubard agitrubard left a comment

Choose a reason for hiding this comment

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

@ilkerCelimli Eline sağlık, buradaki geliştirmeler eksiksiz fakat bizim unit test için kullandığımız token'ları son hâliyle güncellememiz daha iyi olabilir. Onları da güncelledikten sonra mergeleyebiliriz.

@ilkerCelimli
Copy link
Contributor Author

ilkerCelimli commented Jun 4, 2023 •
edited
Loading

@agitrubard Postmandaki tokenleri mi diyorsunuz ? Unit testler sorunsuz çalışıyor şuan,

@agitrubard
Copy link
Collaborator

@agitrubard Postmandaki tokenleri mi diyorsunuz ? Unit testler sorunsuz çalışıyor şuan,

Unit Test'lerde tanımlı olan token'lardan bahsediyorum. Testleri etkileyen bir durum değil fakat oradaki token'ların güncel olması gerektiğinden düzeltilmesi gerekiyor.

@agitrubard
Copy link
Collaborator

@ilkerCelimli Pull request başlığına Feature/ gibi bir prefix eklememize ihtiyacımız bulunmuyor, böyle bir standardımız da yok. Kaldırabilir miyiz?

@ilkerCelimli Buradaki yorumuma uygun olarak pull request başlığını düzeltebilir miyiz?

@agitrubard
Copy link
Collaborator

agitrubard commented Jun 4, 2023 •
edited
Loading

@ilkerCelimli Ek olarak issue Board üzerinde ilerletilmemiş ve label güncellenmemiş, ilgili sütuna ilerleterek label'ı güncel tutabilir miyiz?

@ilkerCelimli ilkerCelimli changed the title Feature/moving the tokentype field from the body section to the header section of the jwt moving the tokentype field from the body section to the header section of the jwt Jun 4, 2023
@ilkerCelimli
Copy link
Contributor Author

@agitrubard Hallettim.

@agitrubard
Copy link
Collaborator

agitrubard commented Jun 6, 2023 •
edited
Loading

@ilkerCelimli conflict oluşmuş, kontrol edebilir misin? GitHub üzerinden çözmeyelim, IntellIJ üzerinden çözelim conflictleri

@ilkerCelimli
Copy link
Contributor Author

İntellij'den halledemedim nedense, ama sanırsam test tokenlerini değiştirdiğiniz için conflict çıkmış

@agitrubard
Copy link
Collaborator

agitrubard commented Jun 6, 2023 •
edited
Loading

İntellij'den halledemedim nedense, ama sanırsam test tokenlerini değiştirdiğiniz için conflict çıkmış

@ilkerCelimli birlikte inceleyelim IDE üzerinden çözelim, olur mu? GitHub üzerinden çözünce hatalara sebebiyet verebiliyor.

@ilkerCelimli
Copy link
Contributor Author

İntellij'den halledemedim nedense, ama sanırsam test tokenlerini değiştirdiğiniz için conflict çıkmış

@ilkerCelimli birlikte inceleyelim IDE üzerinden çözelim, olur mu? GitHub üzerinden çözünce hatalara sebebiyet verebiliyor.

@agitrubard. Tabi müsait bir zamanınızda dcden mesaj atabilirseniz pair yapabiliriz

@ilkerCelimli
Merge remote-tracking branch 'origin/main' into feature/moving-the-to…
7070b21 
…kentype-field-from-the-body-section-to-the-header-section-of-the-jwt

# Conflicts:
#	src/test/java/com/ays/AbstractUnitTest.java
Rapter1990
Rapter1990 approved these changes Jun 7, 2023
View reviewed changes
Copy link
Contributor

@Rapter1990 Rapter1990 left a comment

Choose a reason for hiding this comment

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

LGTM

@Rapter1990 Rapter1990 merged commit 3e5b3c2 into main Jun 7, 2023
@Rapter1990 Rapter1990 deleted the feature/moving-the-tokentype-field-from-the-body-section-to-the-header-section-of-the-jwt branch June 7, 2023 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rapter1990 Rapter1990 Rapter1990 approved these changes

@agitrubard agitrubard agitrubard approved these changes

@skayikci skayikci Awaiting requested review from skayikci skayikci is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Moving The tokenType Field From The Body Section To The Header Section Of The JWT
3 participants
@ilkerCelimli @agitrubard @Rapter1990