Karol Wójcik

Results 38 comments of Karol Wójcik

@weavejester @quatrix @chrysophylax Would you please provide a minimal repro to this? I'm happy to do a PR with the fix :)

For completeness, I'm attaching the full stack of what happens: 1. Server hits the swagger-ui ring handler https://github.com/metosin/reitit/blob/master/modules/reitit-swagger-ui/src/reitit/swagger_ui.cljc#L40 2. Reitit uses `response/resource-response` under the hood https://github.com/metosin/reitit/blob/master/modules/reitit-ring/src/reitit/ring.cljc#L237 3. https://github.com/ring-clojure/ring/blob/bc87e3b68b65f5cc2cb3a622239912d0fa575002/ring-core/src/ring/util/response.clj#L337 4. https://github.com/ring-clojure/ring/blob/bc87e3b68b65f5cc2cb3a622239912d0fa575002/ring-core/src/ring/util/response.clj#L355...

Thank you so much @weavejester. I have updated the patch. Hope the changes match `Ring` standards :) BTW: Thank you for making Ring!

@weavejester Is there something I can do more to have it merged?

Don't sorry, it's all good! I'm patient :) Working demo: [here](https://9g7a59k254.execute-api.eu-central-1.amazonaws.com/api-docs/index.html) Source code: [here](https://github.com/FieryCod/holy-lambda-ring-adapter/tree/0.1.0/examples/native) I have included the patch in my library (see [here](https://github.com/FieryCod/holy-lambda-ring-adapter/blob/master/src/fierycod/holy_lambda_ring_adapter/impl.cljc#L22)). **How to test if patch is...

Hi @weavejester! Sorry for not responding for so long. Here is the [repro/example](https://github.com/FieryCod/ring-graalvm-resource-447). Let me know if it's minimal enough. I've also found this issue: https://github.com/ring-clojure/ring/issues/370, which I was not...

This PR is not yet ready to merge. @borkdude found a corner case. When serving a resource directory e.g. `public` at `/public` and trying to access the `/public` the `resource-data`...

Is there any progress in making tide work correctly with tramp?

@jfromaniello ?

@agorgl @skynet-gh I forgot to create an issue on the GraalVM repo. I will try to find some time this week to address it, but I don't promise anything :joy: