Yimaru/Yimaru-BackEnd
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "Normalize ArifPay checkout payment URLs before returning."

Browse Source 
This reverts commit cd4e3b7811.
This commit is contained in:
Yared Yemane 2026-05-28 03:16:21 -07:00
parent cd4e3b7811
commit 71e605a07a
1 changed files with 1 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
internal/services/arifpay/service.go
View File

@ -8,8 +8,6 @@ import (
"fmt" "fmt"
"io" "io"
"net/http" "net/http"
"net/url"
"path"
"strings" "strings"
"time" "time"
@ -169,7 +167,7 @@ func (s *ArifpayService) InitiateSubscriptionPayment(ctx context.Context, userID
} }
sessionID := fmt.Sprintf("%v", data["sessionId"]) sessionID := fmt.Sprintf("%v", data["sessionId"])
paymentURL := normalizeExternalURL(fmt.Sprintf("%v", data["paymentUrl"])) paymentURL := fmt.Sprintf("%v", data["paymentUrl"])
// Update payment with session info // Update payment with session info
if err := s.paymentStore.UpdatePaymentSessionID(ctx, payment.ID, sessionID, paymentURL); err != nil { if err := s.paymentStore.UpdatePaymentSessionID(ctx, payment.ID, sessionID, paymentURL); err != nil {
@ -186,22 +184,6 @@ func (s *ArifpayService) InitiateSubscriptionPayment(ctx context.Context, userID
}, nil }, nil
} }
func normalizeExternalURL(raw string) string {
u, err := url.Parse(raw)
if err != nil {
return raw
}
if u.Path == "" {
return raw
}
cleaned := path.Clean(u.Path)
if strings.HasSuffix(u.Path, "/") && !strings.HasSuffix(cleaned, "/") {
cleaned += "/"
}
u.Path = cleaned
return u.String()
}
// ProcessPaymentWebhook handles the webhook callback from ArifPay // ProcessPaymentWebhook handles the webhook callback from ArifPay
func (s *ArifpayService) ProcessPaymentWebhook(ctx context.Context, req domain.WebhookRequest) error { func (s *ArifpayService) ProcessPaymentWebhook(ctx context.Context, req domain.WebhookRequest) error {
// ArifPay verify/webhook payloads are inconsistent: some responses include nonce, others only sessionId. // ArifPay verify/webhook payloads are inconsistent: some responses include nonce, others only sessionId.