JoplinSafariWebClipper icon indicating copy to clipboard operation
JoplinSafariWebClipper copied to clipboard

Safari Extension requires to be granted permissions each time

Open pnutster opened this issue 3 years ago • 7 comments

Running Monterey - Safari: Version 15.3 (17612.4.9.1.5) Joplin: Joplin 2.7.15 (prod, darwin)

Any time when Joplin is quit, and then opened again the Safari Extension requires to be granted Joplin Clipper authorization again.

Not entirely sure if you are still actively maintaining the Safari Clipper (I surely hope you do! Joplin is awesome!) Hopefully you can have a look at this. If you require any assistance, more info, more troubleshooting, screenshots, let me know and I will provide.

Thanks for the hard work on the Safari Clipper so far!

pnutster avatar Mar 17 '22 23:03 pnutster

Funny enough, I was looking at that this week with the Joplin API. I think I know how to fix it, but it will probably be a bit before I can finish it due to other commitments. I'll post again when I have a fix.

cweirup avatar Mar 18 '22 01:03 cweirup

@pnutster I finally got a chance to work on a fix and publish a new release. Let me know if this does or does not work to reduce how often you need to grant authorization.

https://github.com/cweirup/JoplinSafariWebClipper/releases/download/v0.3.0/Joplin-Clipper-v0.3.0.zip

cweirup avatar May 18 '22 03:05 cweirup

Hi Chris,

Thanks for working on this!

I did some testing, but can't get it to work at all!

created a short video: https://support.blueoceaninteractive.com//joplin-extension-optim https://support.blueoceaninteractive.com//joplin-extension-optim

Hope to hear from you soon.

With best regards,

Piet Nutbey

On May 18, 2022, at 12:01 AM, Chris Weirup @.***> wrote:

@pnutster https://github.com/pnutster I finally got a chance to work on a fix and publish a new release. Let me know if this does or does not work to reduce how often you need to grant authorization.

https://github.com/cweirup/JoplinSafariWebClipper/releases/download/v0.3.0/Joplin-Clipper-v0.3.0.zip https://github.com/cweirup/JoplinSafariWebClipper/releases/download/v0.3.0/Joplin-Clipper-v0.3.0.zip — Reply to this email directly, view it on GitHub https://github.com/cweirup/JoplinSafariWebClipper/issues/9#issuecomment-1129516430, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABAF4GIPYLZ6PXPANAY5LQ3VKRMSLANCNFSM5RAJIAUQ. You are receiving this because you were mentioned.

pnutster avatar May 18 '22 16:05 pnutster

Piet,

Thanks for sending the video. I'm not sure what's going on, as I'm not seeing it on my computer.

Can you try try two things for me, if you haven't already?

  1. Can you open a new tab or window, navigate to any random page (best if not in your browser cache), and see what happens?
  2. Can you exit Safari then go back in and try the extension again (with Joplin running)?

Not sure these will solve it, but I've had issues in the past with the extension not loading properly with cached pages.

Assuming that doesn't work, I might have to get a build out that has more logging turned on to see what's happening.

cweirup avatar May 19 '22 01:05 cweirup

I'm now able to recreate the problem, which appears to be with initial grant of permission.

You may want to go back to the previous version until I can get this cleaned up, as it may be a while. I'll probably also "unpublished" this build.

Appreciate your patience with this.

cweirup avatar May 19 '22 02:05 cweirup

Fresh startup of system this morning. Fresh random, never visit page. Same thing, Joplin clipper opens up, and closes.

Perhaps an interesting fact, if I renew the Web Clipper token in Joplin, https://support.blueoceaninteractive.com//Shared-Image-2022-05-19-10-03-00 https://support.blueoceaninteractive.com//Shared-Image-2022-05-19-10-03-00 - technically the Web Clipper needs to be granted permission again. WebClipper in both Chrome and FireFox need to be re-granted permission. It does not ask for re-granting permissions for the Safari WebClipper.

On May 18, 2022, at 10:59 PM, Chris Weirup @.***> wrote:

Piet,

Thanks for sending the video. I'm not sure what's going on, as I'm not seeing it on my computer.

Can you try try two things for me, if you haven't already?

Can you open a new tab or window, navigate to any random page (best if not in your browser cache), and see what happens? Can you exit Safari then go back in and try the extension again (with Joplin running)? Not sure these will solve it, but I've had issues in the past with the extension not loading properly with cached pages.

Assuming that doesn't work, I might have to get a build out that has more logging turned on to see what's happening.

— Reply to this email directly, view it on GitHub https://github.com/cweirup/JoplinSafariWebClipper/issues/9#issuecomment-1130989000, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABAF4GNMU3YMVEHJGD7T5QTVKWOA5ANCNFSM5RAJIAUQ. You are receiving this because you were mentioned.

pnutster avatar May 19 '22 13:05 pnutster

I'm experiencing the same problem.

If there's a specific place to look in the code, I can try to fix it.

jeffabailey avatar May 31 '22 03:05 jeffabailey

@pnutster @jeffabailey I just published a new build (v.0.4.0) that I believe solves the authorization persistence issue. I'd appreciate if you could try it out.

Thanks! Chris

cweirup avatar Feb 04 '23 15:02 cweirup

It works.

jeffabailey avatar Feb 04 '23 21:02 jeffabailey

Works beautifully Chris!

Thanks for putting your time and effort into fixing this!

pnutster avatar Feb 04 '23 21:02 pnutster

Great to hear! Sorry it took so long.

cweirup avatar Feb 05 '23 01:02 cweirup