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

better knowledge of which layer can be loaded or not #43

Open
3nids opened this issue Nov 10, 2019 · 5 comments
Open

better knowledge of which layer can be loaded or not #43

3nids opened this issue Nov 10, 2019 · 5 comments
Labels
enhancement New feature or request sogis Going further requires some input from SOGIS team

Comments

@3nids
Copy link
Member

3nids commented Nov 10, 2019

Looking for "Richtplan", trying to load the layer "Abbaustellen", I get an error.
Here is the URI:

dbname='pub' host=geodb.rootso.org port=5432 authcfg=7vcjx1a key='t_id' srid=2056 type=Polygon table="afu_abbaustellen_pub"."abbaustelle" (geometrie) sql=

Apparently there is no schema afu_abbaustellen_pub in the DB.
Is this an issue on the server or the plugin should not propose this layer as PG?

@andreasneumann
Copy link
Contributor

Hi @3nids

I checked this, and the schema and table afu_abbaustellen_pub.abbaustelle seems to exist.

It loads fine for me. Perhaps this has to do with the fact that you have fewer permissions on the DB than I have with my account?

What behavior do you get in this particular case? You have to wait until it times out or do you get an error message quickly?

@3nids
Copy link
Member Author

3nids commented Nov 13, 2019

No, the error is immediate.
It would just be nice to able to bring a bit more information to the user in such case rather than "layer could not be loaded".

@3nids 3nids added the feedback label Nov 15, 2019
@andreasneumann
Copy link
Contributor

How could we do that? Do you have a suggestion? Do you have enough information in such a case to issue a better error message? If yes, I very much agree - the more information, the better.

@3nids
Copy link
Member Author

3nids commented Nov 22, 2019

We could add to the message something like "You might have missing rights".

The best would be if the service could specify permission groups, and we could tell the user: "hey this layer is in this group, are you sure you have access?"

Or even better, use an authentification on the service and only return what it possible for the logged user.

@andreasneumann
Copy link
Contributor

Sound like a good approach.

We can always improve - both our service and then also the plugin - in future versions.

I am sure that there will be other requests or smaller ideas for improvements in the future.

@3nids 3nids added enhancement New feature or request and removed feedback labels Dec 19, 2019
@3nids 3nids changed the title cannot load a layer as PG better knowledge of which layer can be loaded or not Dec 19, 2019
@3nids 3nids added the sogis Going further requires some input from SOGIS team label Dec 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request sogis Going further requires some input from SOGIS team
Projects
None yet
Development

No branches or pull requests

2 participants