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

Authentication API broken missing documentation #25281

Closed
qgib opened this issue Nov 2, 2017 · 15 comments
Closed

Authentication API broken missing documentation #25281

qgib opened this issue Nov 2, 2017 · 15 comments
Labels
Authentication Related to the QGIS Authentication subsystem or user/password handling Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority Regression Something which used to work, but doesn't anymore

Comments

@qgib
Copy link
Contributor

qgib commented Nov 2, 2017

Author Name: Bernhard Ströbl (Bernhard Ströbl)
Original Redmine Issue: 17384
Affected QGIS version: master
Redmine category:authentication_system
Assignee: Alessandro Pasotti


I cannot convince freshly compiled current master 1ffc3fd to save an authentication database.
Furthermore the Python access to QgsAuthManager via @QgsAuthManager.instance()@ seems broken, too:
error is: @type object 'QgsAuthManager' has no attribute 'instance'@

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Giovanni Manghi (@gioman)


  • priority_id was changed from Normal to High

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Alessandro Pasotti (@elpaso)


Nothing is broken: the API has changed, I'll complete the docs but for now: just do QgsApplication.authManager()

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Giovanni Manghi (@gioman)


  • status_id was changed from Open to Feedback

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Bernhard Ströbl (Bernhard Ströbl)


Allesandro, is @QgsApplication.authManager()@ the recommended method for QGIS3?
But the Python problem is minor, the main problem is, that no auth-db is saved.


  • priority_id was changed from High to Normal

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Bernhard Ströbl (Bernhard Ströbl)


  • priority_id was changed from Normal to High

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Alessandro Pasotti (@elpaso)


It's the only method: QgsAccessManager cannot be accesed directly anymore.

Can you please open a separate issue for the DB not saved and describe step-by-step how to reproduce it?

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Bernhard Ströbl (Bernhard Ströbl)


seperate issue opened, see #25282

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Giovanni Manghi (@gioman)


Can we close this?

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Bernhard Ströbl (Bernhard Ströbl)


on my behalf we can but should keep in mind to add the info on the site listing api breaks (or do we need a new ticket for that?)

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Alessandro Pasotti (@elpaso)


please leave this open until I fix the documentation, I'll do it quickly, I promise :)

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Alessandro Pasotti (@elpaso)


Add information about QgsApplication.authManager() and that the instance has been removed


  • subject was changed from Authentication broken to Authentication API broken missing documentation

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2017

Author Name: Giovanni Manghi (@gioman)


  • status_id was changed from Feedback to Open

@qgib
Copy link
Contributor Author

qgib commented Nov 3, 2017

Author Name: Alessandro Pasotti (@elpaso)


fixed in da8a5f4


  • resolution was changed from to fixed/implemented

@qgib
Copy link
Contributor Author

qgib commented Nov 6, 2017

Author Name: Alessandro Pasotti (@elpaso)


  • status_id was changed from Open to Closed

@qgib qgib closed this as completed Nov 6, 2017
@qgib
Copy link
Contributor Author

qgib commented Mar 12, 2018

Author Name: Jürgen Fischer (@jef-n)


  • description was changed from I cannot convince freshly compiled current master 1ffc3fd to save an authentication database.
    Furthermore the Python access to QgsAuthManager via @QgsAuthManager.instance()@ seems broken, too:
    error is: @type object 'QgsAuthManager' has no attribute 'instance'@ to I cannot convince freshly compiled current master commit:1ffc3fd to save an authentication database.
    Furthermore the Python access to QgsAuthManager via @QgsAuthManager.instance()@ seems broken, too:
    error is: @type object 'QgsAuthManager' has no attribute 'instance'@

@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority Authentication Related to the QGIS Authentication subsystem or user/password handling Regression Something which used to work, but doesn't anymore labels May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Authentication Related to the QGIS Authentication subsystem or user/password handling Bug Either a bug report, or a bug fix. Let's hope for the latter! High Priority Regression Something which used to work, but doesn't anymore
Projects
None yet
Development

No branches or pull requests

1 participant