publiclab / Leaflet.DistortableImage

A Leaflet extension to distort or "rubber sheet" images
https://publiclab.github.io/Leaflet.DistortableImage/examples/
BSD 2-Clause "Simplified" License
265 stars 284 forks source link

WebSocket connection to 'ws://localhost:8081/ws' failed #1323

Open jpoep opened 1 year ago

jpoep commented 1 year ago

Describe the bug: Whenever a page that has this package included is opened, the browser repeatedly attempts to connect to the WebPack dev server at localhost:8081. As I'm not using WebPack, this request constantly fails and it also happens in production. When you open your browser's dev tools, you can observe this behavior in the console. This also happens on the demo site.

grafik

It looks like the package is distributed with a development build, but I haven't looked too far into it. I'm not that familiar with WebPack.

Reproduce the behavior:

Browser, version, and operating system:

Any help is much appreciated, thanks a lot in advance! :)

welcome[bot] commented 1 year ago

Thanks for opening your first issue here! This space is protected by our Code of Conduct - and we're here to help. Please follow the issue template to help us help you 👍🎉😄 If you have screenshots or a gif to share demonstrating the issue, that's really helpful! 📸 You can make a gif too! Do join our Gitter channel for some brainstorming discussions.

JoshyB commented 1 year ago

I'm having this same issue, and I'm wondering if a solution has been found?

SashaLifashkin commented 1 year ago

Yeah! I'm having this same issue too.

rodrigore commented 1 year ago

Same issue

AntoineB05 commented 1 year ago

same issue too

luke-z commented 1 year ago

It seems like the problem has been fixed with the release 0.21.10 which somehow isn't available on npm.

The website you linked (https://publiclab.github.io/Leaflet.DistortableImage/examples/index.html) doesn't show the error anymore.

jpoep commented 1 year ago

Interesting, thanks for the update! Any idea who is responsible for publishing 0.21.10 on npm? I bet it's where 99% of people pull this library from and it's already been 3 months since the release.

makrishna commented 1 year ago

It seems like the problem has been fixed with the release 0.21.10 which somehow isn't available on npm.

The website you linked (https://publiclab.github.io/Leaflet.DistortableImage/examples/index.html) doesn't show the error anymore.

Looks like it is not working with websocket builds, can someone have a work around for it.

tap2k commented 1 year ago

Im still having the same issue with 0.21.10 working with NextJS v13.

tap2k commented 1 year ago

Did anyone figure out a solution to fix this?

Parrryy commented 11 months ago

Yeah still having this issue.

Mustafa-Hersi commented 11 months ago

have anyone found a solution to this?

yuri-karelics commented 11 months ago

looks like the only way to fix that is to clone repository locally and make the production build with "npm run build" (I had to disable tests also in gruntfile, they didn't start). When files can be copied into your project from 'dist' folder instead of using NPM package. But keep in mind the latest version of the code in the repository has at least two bugs:

  1. this fix should be reverted, it is wrong https://github.com/publiclab/Leaflet.DistortableImage/commit/21ad3c788e385a8998fcbec392843e745ebd3701
  2. tooltip support implemented wrong, so if you don't need tooltip it will anyway be shown (empty one). Can be fixed with additional condition that 'this.tooltipText' is not empty before attaching the tooltip

If you don't want to do it yourself, just take from 'dist' folder in this fork https://github.com/Karelics/Leaflet.DistortableImage

P.S. maintainers (@jywarren ?) some unexplainable hell is happening with the repository, development build released into NPM, unstable updates merged into main branch, so new release can't be done without adjestments, ... In general your library looks so cool, but what happened in the last year, could you stabilize the situation, thank you!