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

Revert analyzer warning #354

Closed
wants to merge 1 commit into from
Closed

Conversation

erikvanoosten
Copy link

@erikvanoosten erikvanoosten commented Sep 30, 2023

This reverts the change made in 4734d8f because it causes a null pointer dereference.
Fixes #351.

The only change in authpass/lib/cloud_storage/webdav/* since the previous authpass version is this commit. Most probably reverting this commit will fix the issue.

This reverts the change made in 4734d8f because it causes a null pointer dereference.
Fixes authpass#351.
@CLAassistant
Copy link

CLAassistant commented Sep 30, 2023

CLA assistant check
All committers have signed the CLA.

@hpoul
Copy link
Collaborator

hpoul commented Sep 30, 2023

thanks for the report and diving into the problem.. to avoid the deprecation warning i think it should also work to directly access innerText instead of text.. maybe you could try my latest commit in the main branch? a309d20

@erikvanoosten
Copy link
Author

Thanks for looking into this. I am not a Flutter/Dart developer. I tried to install the tools but after 1 hour of installing problems I gave up. I was really surprised you need to install so much stuff. You even have to install malware (Chrome)!

@erikvanoosten
Copy link
Author

erikvanoosten commented Oct 1, 2023

Is it possible for you build a new binary so that I can try it out? I can try it on Ubuntu (Intel), Mac (Intel) or Mac (Arm), whatever is easier for you.

@erikvanoosten
Copy link
Author

Closing in favor of a309d20

@erikvanoosten erikvanoosten deleted the fix351 branch October 6, 2023 18:04
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

Successfully merging this pull request may close these issues.

Unable to add WebDAV source on version 1.9.9
3 participants