Error: Operator does not exist: text ->> unknown with latest change on Postgres about laravel-stripe-webhooks HOT 12 CLOSED

Hmm, it looks like the where clause on json is not supported by all databases.

Can you try this implementation in your StripeWebhookProfile instead? This should work.

public function shouldProcess(Request $request): bool
{
    return WebhookCall::all()->where('payload.id', $request->get('id'))->isEmpty();
}

I'm not really sure if this is less performant than the first solution. But at least it should work with all databases without further changes.

from laravel-stripe-webhooks.

@aerni could you take a look at this?

from laravel-stripe-webhooks.

Other option to update column payload to type json- should work, but doing some testing to see.

ALTER TABLE webhook_calls ALTER COLUMN payload TYPE JSON USING payload::JSON;

from laravel-stripe-webhooks.

After altering the column to JSON on my Postgres database, I'm getting this instead:

[{"exception":"[object] (Illuminate\\Database\\QueryException(code: 22P05): SQLSTATE[22P05]:
 Untranslatable character: 7 ERROR:  unsupported Unicode escape sequence
DETAIL:  \\u0000 cannot be converted to text.

from laravel-stripe-webhooks.

Postgres allow to cast text to json on the fly, source

\Spatie\WebhookClient\Models\WebhookCall::whereRaw("payload::json->>'id' = 'id_sent_by_stripe'")->exists()

P.S. - This query is NOT compatible with MySql

It would be best if payload column can be converted to json type
https://github.com/spatie/laravel-webhook-client/blob/master/database/migrations/create_webhook_calls_table.php.stub#L10

from laravel-stripe-webhooks.

@freekmurze Can you take a look at this? I'm not really familiar with all the implications of a change like that.

from laravel-stripe-webhooks.

Personally I'm inclined to reverse the changes of the PR.

from laravel-stripe-webhooks.

I also think that it's probably a good idea to revert the changes in the Spatie\StripeWebhooks\StripeWebhookProfile. Then everyone can implement their own profile to suit their needs. But I wouldn't revert the option to add a custom profile.

from laravel-stripe-webhooks.

I made a quick fix in my project, like this

Update config file with

'profile' => \Spatie\WebhookClient\WebhookProfile\ProcessEverythingWebhookProfile::class,

from laravel-stripe-webhooks.

I also think that it's probably a good idea to revert the changes in the Spatie\StripeWebhooks\StripeWebhookProfile.

Could you send a PR for that?

from laravel-stripe-webhooks.

Dear contributor,

because this issue seems to be inactive for quite some time now, I've automatically closed it. If you feel this issue deserves some attention from my human colleagues feel free to reopen it.

from laravel-stripe-webhooks.

@freekmurze
The payload column is json type now in v3.x and is supported by most databases

https://github.com/spatie/laravel-webhook-client/blob/06ce01b3855df7007ed42d291f2096af0f7b80c0/database/migrations/create_webhook_calls_table.php.stub?_pjax=%23js-repo-pjax-container%2C%20div%5Bitemtype%3D%22http%3A%2F%2Fschema.org%2FSoftwareSourceCode%22%5D%20main%2C%20%5Bdata-pjax-container%5D#L16

Can you consider this PR again?
#87

This would be a great addition to prevent duplicate webhooks.
//cc @aerni

from laravel-stripe-webhooks.