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.

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

garanti sha512 güncellemesi #158

Merged
merged 4 commits into from
Oct 3, 2023
Merged

Conversation

FikretCin
Copy link
Contributor

Sadece 3D için sha512 güncellemesini yaptım, diğer tipleri bozdumu yada onlarada işlem gerekiyor mu ? Rica etsem test edebilir misiniz ?

  • Destek gerekiyor. kısmındaki hash karşılaştırma mantığını çözemedim, desteğinize ihtiyacım var.

Bu geliştirmeyi %100 bitirmek için her konuda destek olabilirim.

@nuryagdym
Copy link
Collaborator

Merhaba @FikretCin ben de neredeyse ayni guncellemeyi yaptim, ancak size mail atmistim calismadi diye, ulasti mi?

Ben denedigimde, MD Status = 7 geliyor "Guvenlik kodu hatali" hatasi. Sizin yaptiginiz degisiklikler calisiyor mu?

@FikretCin
Copy link
Contributor Author

Evet çalışıyor gönderdiğiniz örneği inceledim hashlenecek veriye currency eklememişsiniz sanırım. 512 methodunda currency (949) da istiyor.

terminalId + orderid + amount + currency + okurl + failurl + islemtipi + taksit + storekey + SecurityData
30691298 + 20230930A815 + 101 + 949 + ...
$secure3dhash = strtoupper(hash('sha512', '3069129820230930A815101http://localhost/garanti/3d/response.phphttp://localhost/garanti/3d/response.phpsales123456781639636D00AB5EF0B3CE073BB222BFAAC2C2C38D'));

@nuryagdym
Copy link
Collaborator

Hmm, bankanin mesajinda hash datasi degismedi sadece algoritmasi degisti diye yazmislar, ondan currency ekleyip eklemediklerine bakmadim.

/**
* Destek gerekiyor.
*/
// if (!$this->requestDataMapper->getCrypt()->check3DHash($this->account, $request->all())) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Aksam bakarim bu kisma, guvenlik amacli bu kontrolun basarili bir sekilde yapilmasi gerekiyor.

Copy link
Collaborator

Choose a reason for hiding this comment

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

burasini artik calisiyor, eski halini getiriniz

composer.json Outdated Show resolved Hide resolved
… getirilidi, yeni versiyon parametresi eklendi.
@FikretCin
Copy link
Contributor Author

Hmm, bankanin mesajinda hash datasi degismedi sadece algoritmasi degisti diye yazmislar, ondan currency ekleyip eklemediklerine bakmadim.

Haklısınız, mailde "Hesaplanan data içerisinde bir değişiklik bulunmuyor." yazmışlar ben de şans eseri fark ettim, commentlerinizi düzeltmeye çalıştım. Farklı yapmamı istediğiniz birşey varsa düzeltmeleri gerçekleştirip destek olmak isterim.

@nuryagdym
Copy link
Collaborator

nuryagdym commented Oct 2, 2023

Merhabalar, guncellemeler ekledim, son halini test eder misiniz, bende calisiyor.

sizde de calisiyorsa,

  1. unit testleri de gunceller misiniz, ondan sonra merge edeyim.
  2. phpstan hatalarini ./vendor/bin/phpstan --generate-baseline komutu ile duzeltebilirsiniz. bu komut phpstan-baseline.neon dosyayi gunceller. onu da commitlersiniz.

Mumkunse bankadan iade, iptal ve non secure odemeyi destekleyen sanal pos hesap bilgileri ister misiniz. Sizin gonderdiginiz hesap bilgileri bende de var. Ama onda, eski hash algoritmayla da, bu 3 islemi yapamiyorum, sonuc olarak yaptigimiz guncelleme onlarda calisir mi emin olamiyorum.

@FikretCin
Copy link
Contributor Author

Yaptığınız değişiklikleri inceledim elinize sağlık, 3d, 3d pay, non-secure ödeme çalışıyor. İptal ve iade de sanırım farklı bir durum var bankadan yeni kullanıcı ve döküman talebinde bulunduk. Test konusunda çok fazla tecrübem yok yapmam gerekenleri iletirseniz sevinirim.

@FikretCin
Copy link
Contributor Author

Garantiden dönüş yaptılar. Artık bireysel test hesabı iletmiyorlarmış. Dev portalı açmışlar dilediğimiz tüm bilgilere buradan ulaşabiliyormuşuz.

https://dev.garantibbva.com.tr/

@nuryagdym
Copy link
Collaborator

Tmm iptal ve iade icin test hesap bilgileri geldiginde bana gonderir misiniz, onlari da test edip merge ederim. Testleri sonradan kendim hallederim

@FikretCin
Copy link
Contributor Author

Tmm iptal ve iade icin test hesap bilgileri geldiginde bana gonderir misiniz, onlari da test edip merge ederim. Testleri sonradan kendim hallederim

Bu linkte test bilgileri var fakat kötü bi haberim var bizdeki bilgilerin aynısı :(

7000679
PROVRFN
123qweASD/

@nuryagdym
Copy link
Collaborator

Bu linkte test bilgileri var fakat kötü bi haberim var bizdeki bilgilerin aynısı :(

Tmm tesekkurler, bankaya bu sorun ile ilgili yazdim. Keske her banka bu sekilde acik kaynak yapsa bilgileri. Bazilari sadece sanal pos musterisi istedikce veriyor.

@nuryagdym
Copy link
Collaborator

Sevdim garantiyi, hemencik cevap verdiler, farkli test kart gonderdiler, o kartla calisti tum islemler.

@nuryagdym nuryagdym merged commit 44ea8c0 into mewebstudio:master Oct 3, 2023
0 of 2 checks passed
nuryagdym added a commit that referenced this pull request Oct 3, 2023
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.

2 participants