clerk-sdk-ruby icon indicating copy to clipboard operation
clerk-sdk-ruby copied to clipboard

Clerk messing up post requests

Open sudhanshug16 opened this issue 1 year ago • 4 comments

We have a Rails app with clerk setup for our admin dashboard which uses ActiveAdmin. When we submit an update/create form, sometimes (~25%) the form data is lost and the request goes to the GET endpoint instead of the POST endpoint. This is not related to ActiveAdmin because we recently added Blazer, which is unaffected by ActiveAdmin and we are seeing the same behavior.

On a side note, the design of this library is very odd. Any chance we can get it to resemble devise? which is pretty much the standard.

sudhanshug16 avatar Jun 05 '24 20:06 sudhanshug16

Hello @sudhanshug16 Could you provide a minimal reproduction repo with the described issue? This would help us identify that the issue is related to the clerk-sdk-ruby, provide a fix and assert that it resolves your issue sooner.

On a side note, the design of this library is very odd. Any chance we can get it to resemble devise? which is pretty much the standard.

That's a nice suggestion! Could you open a separate issue for this to investigate it and provide you with a response?

dimkl avatar Jun 06 '24 10:06 dimkl

Here is a minimal reproducible example: https://github.com/sudhanshug16/clerk-sdk-bug. The issues arise when we add c.middleware_cache_store, especially when the middleware redirects to the sign in url for an instance (presumably to get a new token?) and loses the POST data.

sudhanshug16 avatar Jun 06 '24 15:06 sudhanshug16

any updates? been a week

sudhanshug16 avatar Jun 13 '24 17:06 sudhanshug16

@sudhanshug16 sorry but i didn't have the time to check it yet. I will add a reminder to try check it out ~~by the end of the week~~. Edit: I am not yet available to take a look at this. I will ping you when i managed to find some time.

dimkl avatar Jun 25 '24 13:06 dimkl

Given the length and number of releases since this issue was filed, I'm going to close it out. Please feel free to re-open if the issue still persists.

tmilewski avatar Jan 29 '25 20:01 tmilewski