Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Annotator broken after update to iPadOS 16.3.1 and iOS 16.3.1 (also broken when updated to 16.3) #289

Open
msackeygh opened this issue Jan 23, 2023 · 40 comments

Comments

@msackeygh
Copy link

msackeygh commented Jan 23, 2023

iPadOS 6.3 and macOS 13.2 was just updated today by Apple. Since then, I have been unable to enable Annotator: can't annotate, and can't view PDFs, etc.

@msackeygh msackeygh changed the title Unable to enable after iPadOS 6.3 update Unable to enable after iPadOS 6.3 update and macOS Ventura 13.2 update Jan 23, 2023
@msackeygh
Copy link
Author

msackeygh commented Jan 24, 2023

I have been able to re-enable the plugin, but it still doesn't work.

I re-enabled it by first deleting the plug in, re-installing it (which took a few moments, slower than usual) and then enabling it. However, Annotator still cannot read PDFs. When annotating, the PDF shows up as blank.

@aladmit
Copy link
Collaborator

aladmit commented Jan 24, 2023

@msackeygh do you see any errors in console? What obsidian and obsidian installer version do you use?

I updated my Mac up to 13.2 and didn't get any problems. Everything works as usual. Didn't test on iPad though.

I use obsidian 1.1.12, installer version 1.1.9

@msackeygh
Copy link
Author

@msackeygh do you see any errors in console? What obsidian and obsidian installer version do you use?

I updated my Mac up to 13.2 and didn't get any problems. Everything works as usual. Didn't test on iPad though.

I use obsidian 1.1.12, installer version 1.1.9

I wish I knew how to answer your question regarding what installer version I'm using. I'm using whatever the version of Obsidian is latest in the App store for iOS and iPadOS. On the iOS, Obsidian is in version 1.4.1; plug-in for Annotator is indicated as v0.2.10

I tried Obsidian on my Mac yesterday and I seem to recall it did not have problems, but honestly I can't fully recall and will have to re-test when I get home later tonight. I was testing so much on the iPadOS I can't recall what I did on the Mac.

@msackeygh
Copy link
Author

@aladmit , in checking my Mac version, I'm running Obsidian v1.1.9 and installer version 1.1.9. Annotator (v 0.2.10) is working on macOS 13.2.

However, it is not working on iPadOS 16.3 or iOS 16.3

@aladmit
Copy link
Collaborator

aladmit commented Jan 24, 2023

@msackeygh Great, at least Mac version works) I've tested on iOS 16.3 and got the same problem 😭 I guess it has something to do with WebKit changes.

I'll fix this asap, but I'm not sure how long it would take me to figure out why it happens. Best work around at the moment is to downgrade iOS to 16.2

@aladmit aladmit changed the title Unable to enable after iPadOS 6.3 update and macOS Ventura 13.2 update Unable to enable after iPadOS 16.3 and iOS 16.3 update Jan 24, 2023
@msackeygh
Copy link
Author

@aladmit , thanks! Looking forward to the update.

@aladmit
Copy link
Collaborator

aladmit commented Jan 27, 2023

No progress yet. I see that epub and web annotation starts and only pdf reader fails. Maybe it has something to do with PDF.js that hypothesis use. Unfortunately I can't reproduce it on desktop or get any relevant errors, just guessing.

@msackeygh
Copy link
Author

@aladmit, thanks for the update. Is there something on my end when using the iPad that I could forward? Is there some kind of log I can download and send?

@aladmit
Copy link
Collaborator

aladmit commented Feb 1, 2023

@msackeygh Unfortunately, obsidian on iOS has zero tools for debugging. No logs, no console, no dumps or breakpoints. So I'm just making a guess and break something to see if I'm right🤡

@aladmit
Copy link
Collaborator

aladmit commented Feb 4, 2023

Still no progress. Tried to log all exceptions to a file with onerror events on main window and readers iframe, but there is no exceptions :(

@msackeygh
Copy link
Author

@aladmit, thanks for continuing to try!

@j-xzy
Copy link

j-xzy commented Feb 11, 2023

have same problem

@msackeygh
Copy link
Author

I can confirm that with the update to iOS and iPadOS 16.3.1, Annotator is still broken.

@msackeygh msackeygh changed the title Unable to enable after iPadOS 16.3 and iOS 16.3 update Annotator broken after update to iPadOS 16.3.1 and iOS 16.3.1 (also broken when updated to 16.3) Feb 14, 2023
@fritzcloud
Copy link

Doesn't work on ipados 16.4 beta

@aladmit
Copy link
Collaborator

aladmit commented Feb 25, 2023

Unfortunately, I'm stuck and I don't know what else to try :(

@elias-sundqvist Could you have a look on this? Maybe you would see something I don't.

@patrickongwong
Copy link

Hi @aladmit, thanks so much for the updates on this. I'll downgrade my iOS for now. Nothing is more important in my iPad than Obsidian and the Obsidian annotator. It's such a great product and plug-in combination.

Thank you so much.

@elias-sundqvist
Copy link
Owner

I'd be happy to take a look, but I don't have an iOS device to test on 😕

@fritzcloud
Copy link

I'd be happy to take a look, but I don't have an iOS device to test on 😕

Can I help with testing on my iPad? I can't code but I want to help.

@msackeygh
Copy link
Author

Unfortunately, I'm stuck and I don't know what else to try :(

@elias-sundqvist Could you have a look on this? Maybe you would see something I don't.

Thank you for trying. I keep checking back every so often to see if a fix has been made. Also, I'm not able to downgrade (and wouldn't want to for security reasons). I believe Apple has stopped signing any iOS lower than 16.3.1

@VileEnd
Copy link

VileEnd commented Mar 2, 2023

looks like the file can't be found (maybe how its beeing referenced has been changed?) (dont really have more time atm, sry) but here the console log from the ipad; hope that helps; ( thx xcode)
err404

@aladmit
Copy link
Collaborator

aladmit commented Mar 3, 2023

@VileEnd how did you get this log?

@VileEnd
Copy link

VileEnd commented Mar 3, 2023

@VileEnd how did you get this log?

@aladmit

I used this plugin here:
https://github.com/KjellConnelly/obsidian-dev-tools

And added it to my .obsidian/plugin dir
Other way is to do it with xcode but that's a bigger hustle (or use xcode and use in there the plugin so you don't have to own an iPad)

@aladmit
Copy link
Collaborator

aladmit commented Mar 5, 2023

@VileEnd Thanks for helping with this! I investigated this error, but unfortunately it doesn't have anything to do with iOS problems. Hypothesis tries to download favicon, but gets 404 and returns the error. I commented favicon requests out, error went away, but problem with iOS stayed the same :(

Looks like we will struggle with this for a while. Added this as known issue to README.

@aladmit
Copy link
Collaborator

aladmit commented Mar 5, 2023

I have a new hypothesis on this. I found that there is a canceled network request. It's possible to see it on desktop too.

Hypothesis makes request to https://hypothes.is/app.html, but browser cancels it because of strict-origin-when-cross-origin.

iOS 16.3 update brought a few related security fixes:
1.Fixed Cross-Origin-Embedder-Policy incorrectly blocking scripts. https://developer.apple.com/documentation/safari-release-notes/safari-16_3-release-notes#Resolved-Issues
2. Fixed: Check for strict-dynamic in script-src and default-src directives. https://developer.apple.com/documentation/safari-release-notes/safari-16_3-release-notes#Resolved-Issues

So. Maybe webview on iOS blocks whole iframe with plugin because of strict policy violation 🤔

Plugin creates iframe and catches in it all fetch requests to hypothes.is and replaces responses with local data. But misses this one because it goes from iframe created by hypothesis boot script >.<

@elias-sundqvist do you see a way to catch this request too? I believe you know about catching fetch requests much more than me.

@fritzcloud
Copy link

fritzcloud commented Mar 8, 2023

Getting a new error on IOS 16.4

IMG_0311

Edit: Nevermind that error, my fault. It seems to be fixed on iPadOS 16.4 beta now. Annotator works again.

@aladmit
Copy link
Collaborator

aladmit commented Mar 10, 2023

Huh, that's weird. But it would be great if problem would fix itself with 16.4 release😄

@baseliners
Copy link

In case this helps - I'm able to open the annotation view on my iPad (16.3) for an epub. However, the UI is messed up when I open the annotation sidebar on the iPad (see screenshot). Works fine on my Mac though.

Was thinking of converting PDF to epub for use on mobile. Maybe fixing the sidebar issue on mobile is easier?

IMG_2D99FB629A0A-1

@patrickongwong
Copy link

I'm confirming it works with iPadOS 16.4.

It seems the best is for everyone to just upgrade to (if Beta is acceptable) or wait to upgrade this version.

Thank you so much for the effort @aladmit @elias-sundqvist

@baseliners
Copy link

Thx @patrickongwong ! I installed the 16.4 Beta and can also confirm that it's working!

One thing - the resolution is quite low so when I zoom in, the text is very pixelated. I don't see the same issue on the Mac (for the same file) and also tried opening the same PDF using PDFExpert on my iPad and that also renders fine. Not sure if this is new behavior or if it's always been like this on mobile? Any way to get the full res view in Annotator?

@aladmit aladmit pinned this issue Mar 23, 2023
@msackeygh
Copy link
Author

Welp, iPadOS 16.4 was released today to the general public and it looks like Annotator is working again in this platform! At least this seems to be so with preliminary testing :)

@maizehsu
Copy link

Hooray! Here to confirm that annotator does work again after updating to IPadOS 16.4.1. 🎉🎉

Welp, iPadOS 16.4 was released today to the general public and it looks like Annotator is working again in this platform! At least this seems to be so with preliminary testing :)

@jfs
Copy link

jfs commented Apr 14, 2023

Don’t forget to update the README.

@Pchatain
Copy link

Pchatain commented Sep 3, 2023

This plug in does not work as far as I can tell on IOS 16.6, iPhone 12. False alarm! I forgot to add the setting to download pdf’s from sync.

@fritzcloud
Copy link

It‘s also broken on iPadOS 17 stable

@LuGHuaaa
Copy link

Any news or updates?

@jonas886
Copy link

jonas886 commented Nov 2, 2023

IT WORKS !

I am in M1 and macos 14.0 (23A344)

@deekayone
Copy link

I have macOS 14.0 and annotator works great.
I have iPadOS 16.6 and annotator does not work.
Is this the current state? Or have some found a way to make iPadOS16.6 work?

@891680996
Copy link

does not work on iPadOS 17.1

1 similar comment
@DrKitsune14
Copy link

does not work on iPadOS 17.1

@ess-seb
Copy link

ess-seb commented Feb 18, 2024

Same here – iPad Pro M1 iPadOS 17.4 beta, Annotator doesn’t work :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests