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

New version iVim 2.29 (5) broke opening an encrypted file #189

Closed
DanielMontreal opened this issue Feb 13, 2021 · 15 comments
Closed

New version iVim 2.29 (5) broke opening an encrypted file #189

DanielMontreal opened this issue Feb 13, 2021 · 15 comments

Comments

@DanielMontreal
Copy link

Since this new version, opening an encrypted file from iCloud-Drive does not show the encryption dialog, instead it opens an empty file in vim.

The previous version was opening a dialog to ask for the password, then opened the file correctly decoded.

Thanks!

Daniel

@terrychou
Copy link
Owner

Thanks for the feedback!

What about other non-encrypted files? Can they be opened as expected this way? Which iOS version and device are you using?

@DanielMontreal
Copy link
Author

Oups! Sorry, I feel like a beginner!

Non-encrypted files open normally, as expected. The bad behaviour is only with encrypted files.

Iphone 6s / IOS 14.4

Thanks

@terrychou
Copy link
Owner

I have made some related fixes in version 2.29(6) and pushed it to TestFlight. Let's see whether it solves this issue or not.

@DanielMontreal
Copy link
Author

No it does not solve the problem. Same behaviour.

Made a little video of the screen capture. Usually when I click iVim to open the files, there is a dialog asking me for the password. Now, nothing.

RPReplay_Final1613314249.MP4

@terrychou
Copy link
Owner

After iVim is opened, what is the file name of the "empty" buffer? (Use ctrl+G to see)

Could you please try this: copy a encrypted file into iVim, then try to open it 1) with command ":edit"; 2) from the file browser?

@DanielMontreal
Copy link
Author

ctrl-G shows:
"[No Name]" --No lines in buffer

opening with command :edit (or :open) and the name of the file shows the old dialog to put the password with this:
Warning: Using a weak encryption method;
Enter encryption key:
and if I enter the correct password, the file opens but everything is garbage like if there is no decryption at all (like when if wasn't working correctly last month).

if I open from the file browser of the file already on the iphone: it opens an empty file and the ctrl-G shows:
"[No Name]" --No lines in buffer

Thanks

@terrychou
Copy link
Owner

By "file browser", I mean the newly added "file browser" view (":h ivim-file-browser"), rather than the iOS's Files.app. Use ":ibrowse -w" to open one.

Then it looks like the password input prompt is not showing on your iPhone. Maybe it has something to do with the way iPhones work with the multiple windows mechanism...

@DanielMontreal
Copy link
Author

":ibrowse -w" does not work at all, but ":ibrowse" does open a browser to choose files. With that method, it works perfectly: it shows the password dialog (like previous sub-version) and it opens the file correctly (decoded).

All other methods fail. So I will use this method for the moment until the other works.

@terrychou
Copy link
Owner

I've pushed version 2.29(7) to TestFlight. I made some other attempts to "possibly" deal with this issue, please give it a try.

The problem right now is that, I don't have an iPhone by hand. Therefore, I can only test it on iPads and in the iPhone simulators. However, I've never succeeded to re-produce this issue. As a result, all the attempts are based on my guesses.

@DanielMontreal
Copy link
Author

I can do the tests that you need. I can even make a video of my screen.

The new version is worse than the previous.
Same behaviour if I try to open the file like usually (from the icloud then opening ivim, see previous video).
But now the :ibrowse way that was working before does not even work anymore. It shows me the password dialog for about half a second then closes it and opens a blank file.

what test you need me to do?

@terrychou
Copy link
Owner

Thank you. According to your feedback so far, I think I know what is wrong. But I just don't know where I should place the fix. No worry, those attempts didn't fail for nothing. They have been narrowing possible targets for me.

As to the new :ibrowse problem, I am sure what is wrong. I'll fix it.

I will do some more fixes and push the update to TestFlight soon. Please help test it.

@DanielMontreal
Copy link
Author

IT WORKS! with the new version as of this morning, it works! opening the file either from the icloud or from the vim browser.

Minor glitch: from the vim brower (":ibrowse") the password window closes then it opens again. Here is the video of this behaviour if interested.
THANKS it's great!

RPReplay_Final1613747346.mov

@terrychou
Copy link
Owner

Great! Then I will publish it to the App Store after some more refinements.

I am also aware of the glitch, and I will improve it. By the way, the ":edit" problem the encrypted file has also been solved (actually in the last build).

@DanielMontreal
Copy link
Author

The new version you 2.30(1) works perfectly now with the encrypted files. Not even a glitch!

Congratulations, you nailed it.

I tested it from the internal browser :ibrowse and also calling it from the file itself (from icloud).

Thanks for everything. Case closed I think.

If you need any more testing, I'm willing. Used vim all my professional life (since 1986). Best editor ever (I teach it also in my System administration class).

@terrychou
Copy link
Owner

Sure thing. Thanks for helping.

As a matter of fact, since I have added you into the testing group, you will receive every update for testing. Please let me know if you find any problem or have any suggestion.

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

2 participants