search

lnurl / luds

lnurl specifications
584 stars 138 forks source link

Access-Control-Allow-Origin #267

Open shocknet-justin opened 1 month ago

shocknet-justin commented 1 month ago

https://github.com/lnurl/luds/issues/259

hsjoberg commented 1 month ago

Why are we adding OPTIONS requests to LNURL? It's an all-covering breaking change mandating all LNURL server implementations to support it.

I suggest we remove this.

AaronDewes commented 1 month ago

Why are we adding OPTIONS requests to LNURL? It's an all-covering breaking change mandating all LNURL server implementations to support it.

I suggest we remove this.

This is necessary for the LNURL to work/be resolved from web browsers.

AaronDewes commented 1 month ago

I was mistaken, LNURL resolution is considered a "simple" request that does not require options.

kaloudis commented 1 month ago

Long needed guidance. Thanks for writing this.

ACK