Issue while payment redirection in some hosting provider due to caching

[knit-pay]

Hello Team

I encountered another rare issue. This happens with websites on some hosting providers. Some hosting providers by default cache the homepage of the website. because of which filter_has_var( INPUT_GET, 'payment_redirect' ) always returns false even if payment_redirect and key are available in the URL.

wp-pay-core/src/Plugin.php

Line 480 in 1d52978

if ( ! filter_has_var( INPUT_GET, 'payment_redirect' ) || ! filter_has_var( INPUT_GET, 'key' ) ) {

I suggested 2 of my clients contact the hosting providers and remove the caching on the homepage, and this solved the issue. But most of such clients think that this is the issue with the plugin instead of hosting. because they don't contact us.

Proposed Solution: If at the function below, we will change the URL format to an SEO-friendly URL for the hosting which supports SEO-friendly URL, this issue will get resolved. If hosting does not support SEO-friendly URL, we can continue to use this method.

wp-pay-core/src/Payments/Payment.php

Line 294 in 1d52978

$url = add_query_arg(

The same issue is there with handle_returns and various other functions also.

What do you think? Can we fix this issue anyhow?

[remcotolsma]

Good catch, we should be able to fix that. In our PayPal gateway integration we (ab)use the REST API for this:

• https://github.com/pronamic/wp-pronamic-pay-paypal/blob/6045abc5eb25ee90a9b5c96dbc01ad1d1bd8bbf0/src/Gateway.php#L162-L178
• https://github.com/pronamic/wp-pronamic-pay-paypal/blob/b0e8e519e1795deef42810178d968b61fa5fc4dc/src/NotificationsController.php#L115-L138
• https://github.com/pronamic/wp-pronamic-pay-paypal/blob/b0e8e519e1795deef42810178d968b61fa5fc4dc/src/NotificationsController.php#L335-L377

The advantage of the WordPress REST API is the ease of registering endpoints which automatically work with 'Plain Permalinks' (Ugly Permalink) and 'Pretty Permalinks':

• https://wordpress.org/support/article/using-permalinks/#plain-permalinks
• https://wordpress.org/support/article/using-permalinks/#pretty-permalinks

We could use an endpoint like /wp-json/pronamic-pay/v1/payments/:payment_id/redirect?key=:key. Downside is that it's not common to output HTML via REST API endpoints (required for 'redirect' via HTML).

Maybe we should split this up too:

flowchart TD
    A[Start] --> B{HTTP or HTML?}
    B -->|HTTP| C[Redirect to gateway]
    B -->|HTML| D[Redirect to WordPress page]
    D --> E[Auto submit HTML form]

Loading

The question then is whether we make use of a regular WordPress page for the HTML 'redirect' or opt for custom rewrite rule. I think we should go for a custom rewrite rule so that we can use 'Pretty Permalinks', something like:

/:namespace/payments/:payment_id/redirect?key=:key