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

PKP: fix by using a specific User-Agent #336

Merged
merged 2 commits into from
Feb 14, 2025
Merged

PKP: fix by using a specific User-Agent #336

merged 2 commits into from
Feb 14, 2025

Conversation

derhuerst
Copy link
Member

We're currently investigating if this actually fixes the problems with the PKP endpoint. There seems to be some IP-based blocking in place.

#184 (comment)

@takenek
Copy link

takenek commented Feb 14, 2025

@derhuerst Last time i repeat mobil.rozklad-pkp.pl is a comercial endpoint same like rozklad-pkp.pl is a comercial webpage etc.. Look there is ads and other - this is not PUBLIC webpage information! Official public Polish railroad information webpage is: https://www.portalpasazera.pl/

Best Regards

@fwam
Copy link

fwam commented Feb 14, 2025

@derhuerst Last time i repeat mobil.rozklad-pkp.pl is a comercial endpoint same like rozklad-pkp.pl is a comercial webpage etc.. Look there is ads and other - this is not PUBLIC webpage information! Official public Polish railroad information webpage is: https://www.portalpasazera.pl/

Best Regards

@takenek - rozklad-pkp.pl is a subject to PKP Informatyka, being owned by PKP SA – a state-owned company, therefore a subject to Open Data Act.

Art. 10 p. 3 of the Open Data Act states the API not only is PUBLIC, it also should be DOCUMENTED, which it is not.

Best Regards, fwam.

Copy link

@fwam fwam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'd replace the UA with Dalvik/2.1.0 in example.js

otherwise works!

@takenek
Copy link

takenek commented Feb 14, 2025

Szanowni bo nie będę się powtarzać.
Używanie w/w narzędzia narusza regulamin korzystania z serwisu oraz API a w szczególności pkt 7,8,9,10,11.
http://regulamin.rozklad-pkp.pl/
Używanie w/w narzędzia spowoduje blokadę dostępu do serwisów.

Pozdrawiam.

Dear all, as I will not repeat myself.
The use of the aforementioned tool violates the terms of use of both the service and the API, particularly points 7, 8, 9, 10, and 11.
http://regulamin.rozklad-pkp.pl/
Using this tool will result in access being blocked to the services.

Best regards.

@fwam
Copy link

fwam commented Feb 14, 2025

Szanowni bo nie będę się powtarzać. Używanie w/w narzędzia narusza regulamin korzystania z serwisu oraz API a w szczególności pkt 7,8,9,10,11. http://regulamin.rozklad-pkp.pl/ Używanie w/w narzędzia spowoduje blokadę dostępu do serwisów.

Pozdrawiam.

Dear all, as I will not repeat myself. The use of the aforementioned tool violates the terms of use of both the service and the API, particularly points 7, 8, 9, 10, and 11. http://regulamin.rozklad-pkp.pl/ Using this tool will result in access being blocked to the services.

Best regards.

Regulamin Serwisu nie jest ważniejszy niż prawo polskie, podałam ci dokładny artykuł xD

Powtórzę, art. 10 p. 3 Ustawy o otwartych danych i ponownym wykorzystywaniu informacji sektora publicznego z dnia 11 sierpnia 2021 r.

@fwam
Copy link

fwam commented Feb 14, 2025

Poza tym, wydaje mi się, czy przypadkiem https://rozklad-pkp.pl/pl/page/regulamin nie jest aktualnym regulaminem? Chyba, że macie dwa xDDDD

@banocean
Copy link

@derhuerst Last time i repeat mobil.rozklad-pkp.pl is a comercial endpoint same like rozklad-pkp.pl is a comercial webpage etc.. Look there is ads and other - this is not PUBLIC webpage information! Official public Polish railroad information webpage is: https://www.portalpasazera.pl/
Best Regards

@takenek - rozklad-pkp.pl is a subject to PKP Informatyka, being owned by PKP SA – a state-owned company, therefore a subject to Open Data Act.

Art. 10 p. 3 of the Open Data Act states the API not only is PUBLIC, it also should be DOCUMENTED, which it is not.

Best Regards, fwam.

as far as I understand what has been pointend out is not regard of availibility of data but place where it is shared, this message seems to point to diferent place as source of same data and says that mentioned apis are internal way to interact for tools/services provided by some of entities being part of Grupa PKP

@DomeQdev
Copy link

Szanowni bo nie będę się powtarzać. Używanie w/w narzędzia narusza regulamin korzystania z serwisu oraz API a w szczególności pkt 7,8,9,10,11. http://regulamin.rozklad-pkp.pl/ Używanie w/w narzędzia spowoduje blokadę dostępu do serwisów.

Pozdrawiam.

Szanowny @takenek, jeśli PKP PLK też tak uważa, niech wyda oficjalne pismo z podpisem osoby która tak uważa. Pozdrawiam ❤️❤️

@fwam
Copy link

fwam commented Feb 14, 2025

@banocean this very endpoint, as a matter of fact, is used publicly by rozklad-pkp.pl phone app, as well as old.rozklad-pkp.pl

@banocean
Copy link

Szanowni bo nie będę się powtarzać. Używanie w/w narzędzia narusza regulamin korzystania z serwisu oraz API a w szczególności pkt 7,8,9,10,11. http://regulamin.rozklad-pkp.pl/ Używanie w/w narzędzia spowoduje blokadę dostępu do serwisów.
Pozdrawiam.

Szanowny @takenek, jeśli PKP PLK też tak uważa, niech wyda oficjalne pismo z podpisem osoby która tak uważa. Pozdrawiam ❤️❤️

@banocean this very endpoint, as a matter of fact, is used publicly by rozklad-pkp.pl phone app, as well as old.rozklad-pkp.pl

The pooint is the fact it's being used by the app, it's not meant for outside data sharing (since everyone prob would like data being shared to broader extent than what's displayed in the app). As pointed out ealier there are other platform and entities that shall provide same information to their full extent

@takenek
Copy link

takenek commented Feb 14, 2025

banocean

Już raz PKP PLK wyjaśniała temat - wystarczy Ci sejmowy dokument - czy tego też nie zrozumiesz jak regulaminu?
https://www.sejm.gov.pl/sejm8.nsf/InterpelacjaTresc.xsp?key=43069581

Pozdrawiam.

@fwam
Copy link

fwam commented Feb 14, 2025

banocean

Już raz PKP PLK wyjaśniała temat - wystarczy Ci sejmowy dokument - czy tego też nie zrozumiesz jak regulaminu? https://www.sejm.gov.pl/sejm8.nsf/InterpelacjaTresc.xsp?key=43069581

Pozdrawiam.

Ustawa jest z 2021, odpowiedź z 2017.

Swoją drogą, proszę o niestosowanie ad personam :)

@takenek
Copy link

takenek commented Feb 14, 2025

Poza tym, wydaje mi się, czy przypadkiem https://rozklad-pkp.pl/pl/page/regulamin nie jest aktualnym regulaminem? Chyba, że macie dwa xDDDD

To regulamin strony rozklad-pkp.pl - wystarczy przeczytać jest napisane czego to regulamin.
Tak mamy regulamin dla stron, API i innych serwisów - to takie dziwne, że mamy wiele regulaminów dla różnych usług wystawionych do świata?

@fwam
Copy link

fwam commented Feb 14, 2025

Poza tym, wydaje mi się, czy przypadkiem https://rozklad-pkp.pl/pl/page/regulamin nie jest aktualnym regulaminem? Chyba, że macie dwa xDDDD

To regulamin strony rozklad-pkp.pl - wystarczy przeczytać jest napisane czego to regulamin. Tak mamy regulamin dla stron, API i innych serwisów - to takie dziwne, że mamy wiele regulaminów dla różnych usług wystawionych do świata?

"Korzystając z serwisów i aplikacji "Rozklad-pkp" akceptujcie Państwo postanowienia niniejszego Regulaminu." - regulamin.rozklad-pkp.pl
"Korzystając z serwisu akceptujcie Państwo postanowienia niniejszego Regulaminu." - https://rozklad-pkp.pl/pl/page/regulamin

wygląda to, jakby strona miała dwa regulaminy.

@takenek
Copy link

takenek commented Feb 14, 2025

Podobnie jak w poprzednim wątku kończę, bo mi ręce opadają, tym bardziej jak widzę "xDDDD", bo mam wrażenie, że z dziećmi piszę.
Grzecznie proszę o nie łamanie regulaminu serwisu i nie stosowanie w/w rozwiązania i nie szukania jak omijać zabezpieczenia przed masowym ściąganiem danych z naszych serwisów. Tym bardziej później za pieniądze udostępnianie tych danych pod swoim szyldem.

Pozdrawiam

@fwam

This comment was marked as off-topic.

@DomeQdev

This comment was marked as off-topic.

@banocean
Copy link

banocean

Już raz PKP PLK wyjaśniała temat - wystarczy Ci sejmowy dokument - czy tego też nie zrozumiesz jak regulaminu? https://www.sejm.gov.pl/sejm8.nsf/InterpelacjaTresc.xsp?key=43069581

Pozdrawiam.

Nie wiem, jak treść odpowiedzi tej interpelacji się odnosi do tego, o czym pisałem. Jedyne, na co zwracałem uwagę to fakt ignorowania przez nas tego, że nie jest to dedykowane api do tych zastosowań (co nie zmienia faktu braku wymaganego ustawowo api) i dalej to może powodować różnego rodzaju problemy np. zmiany dedykowane tylko do aplikacji, które wprowadzają zmiany psujące obecne zastosowania.

Co do wykorzystania tego przez twórców biblioteki to ja nie widzę problemu, bo jednak nie jest to strona biorąca udział w zawarciu jakiejkolwiek umowy tj. akceptowaniu regulaminu, ale gorzej ze wszystkimi używającymi tej biblioteki, by faktycznie zbierać te dane

@banocean

This comment was marked as off-topic.

@banocean

This comment was marked as off-topic.

@DomeQdev

This comment was marked as off-topic.

@derhuerst
Copy link
Member Author

(Responding in English because I don't speak Polish.)

I think we can agree that hafas-client by itself is merely a tool, used to access many different HAFAS endpoints. What is considered in line with or in violation of the terms of service of a particular HAFAS endpoint depends on who uses it and what for.

The current state of access to Polish static (and realtime!) transport data is irrelevant for this PR, just like the question if mobil.rozklad-pkp.pl's terms of service are in line with EU regulations, and just like the question what constitutes mass data retrieval. These are important discussions to have, but please have them somewhere else (e.g. in #opentransport:matrix.org, or in public-transport/ideas).

@derhuerst derhuerst merged commit 7e2ae31 into main Feb 14, 2025
12 of 24 checks passed
@derhuerst derhuerst deleted the fix-pkp branch February 14, 2025 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants