-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Fix #17444 : GeoNode data provider completely ignores authentication #5634
Conversation
…ion - hide the authentication group until GeoNode API supports authentication.
src/gui/qgsnewhttpconnection.h
Outdated
@@ -56,6 +56,7 @@ class GUI_EXPORT QgsNewHttpConnection : public QDialog, private Ui::QgsNewHttpCo | |||
enum Flag | |||
{ | |||
FlagShowTestConnection = 1 << 1, //!< Display the 'test connection' button | |||
FlagHideAuthenticationGroup = 2 << 2, //!< Hide the Authentication group |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be 1 << 2
@@ -20,14 +20,16 @@ | |||
#include "qgsgeonodeconnection.h" | |||
#include "qgsgeonoderequest.h" | |||
|
|||
// For now we are hiding away authentication options since | |||
// there is no support yet for tication in the GeoNode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tication -> authentication
@@ -20,14 +20,16 @@ | |||
#include "qgsgeonodeconnection.h" | |||
#include "qgsgeonoderequest.h" | |||
|
|||
// For now we are hiding away authentication options since | |||
// there is no support yet for tication in the GeoNode | |||
// We will re-enable this when this limitation changes. TS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there a GeoNode ticket number which could be linked here?
Thanks @timlinux! |
Thanks @timlinux, this looks like a good solution. |
Thanks for the inputs @nyalldawson and @elpaso |
For now we deal with this by adding a flag to QgsNewHttpConnection that allows you to hide the authentication group until GeoNode API supports authentication.
With
QgsNewHttpConnection::FlagHideAuthenticationGroup
set (in GeoNode connection dialog):With
QgsNewHttpConnection::FlagHideAuthenticationGroup
omitted (default behaviour) e.g. in WMS connection dialog:Description
The GeoNode API does not yet support authentication through the API, only from the underlying backend. In the future we will probably work for GeoNode to support authentication through their API but for now we are excluding the authentication options which do nothing which is confusing and misleading to users.
Checklist
fixes #11111
in the commit message next to the description[FEATURE]
in the commit message[needs-docs]
in the commit message and contain sufficient information in the commit message to be documentedscripts/prepare-commit.sh
script before each commit