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

Navitia: BlockedException when searching directions #583

Closed
bre75 opened this issue Feb 7, 2019 · 14 comments
Closed

Navitia: BlockedException when searching directions #583

bre75 opened this issue Feb 7, 2019 · 14 comments
Assignees
Labels
upstream 🏞️ Related to an upstream issue.

Comments

@bre75
Copy link

bre75 commented Feb 7, 2019

[
screenshot_20190207-173207

@grote
Copy link
Owner

grote commented Feb 7, 2019

can't reproduce this here with the paris provider

@grote grote added the needs info ℹ️ More information needed, label Feb 7, 2019
@grote grote changed the title Not working since 2.0.5 upgrade Navitia: BlockedException when searching directions Feb 7, 2019
@bre75
Copy link
Author

bre75 commented Feb 7, 2019 via email

@bre75
Copy link
Author

bre75 commented Feb 8, 2019 via email

@jeanrjc
Copy link

jeanrjc commented Feb 19, 2019

Same issue for me since the update.

de.schildbach.pte.exception.BlockedException: https://api.navitia.io/v1/journeys?from=stop_area%3AOIF%3ASA%3A8775863&to=stop_area%3AOIF%3ASA%3A8739100&datetime=20190219T090456&datetime_represents=departure&min_nb_journeys=6&depth=0&walking_speed=1.12
de.schildbach.pte.util.HttpClient.getInputStream(HttpClient.java:292)
de.schildbach.pte.util.HttpClient.get(HttpClient.java:209)
de.schildbach.pte.util.HttpClient.get(HttpClient.java:197)

@grote
Copy link
Owner

grote commented Feb 19, 2019

Does it happen on all searches? Also departures and nearby stations? Also non-saved direction searches?

@jeanrjc
Copy link

jeanrjc commented Feb 19, 2019

It seems to bug when it involves saved (starred) directions. Using history is ok for the few tries I've made.
I can't really try all combinations, so I can't say which one works for sure. But directions I saved are not working, but if I set the same locations by hand and if don't chose the saved locations that are proposed, it works.

@grote
Copy link
Owner

grote commented Feb 19, 2019

Strange. Might be similar to #207. I have no time to investigate this. Maybe somebody with a bit of technical knowledge can do this.

@bre75
Copy link
Author

bre75 commented Feb 19, 2019 via email

@ialokim
Copy link
Collaborator

ialokim commented Feb 19, 2019

Same issue for me since the update.
de.schildbach.pte.exception.BlockedException: https://api.navitia.io/v1/journeys?from=stop_area%3AOIF%3ASA%3A8775863&to=stop_area%3AOIF%3ASA%3A8739100&datetime=20190219T090456&datetime_represents=departure&min_nb_journeys=6&depth=0&walking_speed=1.12

@jeanrjc Where did you get this error from? This url lacks the coverage part, which causes the BlockedException (see Navitia playground). But you are right, after adding the coverage, Navitia returns an invalid_object error (see here).

So there are really two separate issues here:

@jeanrjc
Copy link

jeanrjc commented Feb 19, 2019

It's the exception I copied from the app, as in the screenshot above.
I think it is similar to #207 although I don't have full comprehension on the matter.

@ialokim
Copy link
Collaborator

ialokim commented Feb 21, 2019

@grote While investigating, I noticed AbstractNavitiaProvider does always call the Navitia API for journeys without specifying a coverage. Is that intended behavior? It seems this results in the BlockedException instead of returning an invalid_object on Navitia's side (see the two links to Navitia playground above).

Adding the coverage part would at least prevent this strange error and instead result in a (slightly better understandable) "invalid ID" error for the user.

@grote
Copy link
Owner

grote commented Feb 21, 2019

I'd say it should use the coverage in its requests, but I didn't write this provider, so I am not the authority here. How is this even working without specifying the coverage?

@El-Gavy
Copy link

El-Gavy commented Mar 8, 2019

I have the same issue, here is a screenshot in case it can help :

screenshot_2019-03-08-10-04-36

@ialokim ialokim added upstream 🏞️ Related to an upstream issue. and removed needs info ℹ️ More information needed, labels Jul 21, 2019
@ialokim ialokim self-assigned this Jul 21, 2019
@ialokim
Copy link
Collaborator

ialokim commented Sep 4, 2020

This issue is fixed upstream.

@ialokim ialokim closed this as completed Sep 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream 🏞️ Related to an upstream issue.
Projects
None yet
Development

No branches or pull requests

5 participants