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

NearbyParentFragment : added referer #5417

Merged
merged 2 commits into from
Jan 3, 2024
Merged

NearbyParentFragment : added referer #5417

merged 2 commits into from
Jan 3, 2024

Conversation

rohit9625
Copy link
Contributor

Description (required)
In the file NearbyParentFragment.java, I added a header property, i.e., the referer - http://maps.wikimedia.org/, and set the tile source to Wikimedia.

Fixes #5374

What changes did you make and why?
I added a Referer - http://maps.wikimedia.org/ in the header requests sent when fetching map images in the Nearby section.
This change was needed to prevent us from being banned from the map tiles server.

Also, setTileSource to WIKIMEDIA

Tests performed (required)

Tested betaDebug on Samsung A14 with API level 34.

Screenshots (for UI changes only)
Did not perform any UI changes.

rohit9625 and others added 2 commits January 3, 2024 13:34
In file NearbyParentFragment.java, I added header property, i.e., the referer - http://maps.wikimedia.org/
and set tile source to wikimedia.
@nicolas-raoul nicolas-raoul merged commit 6a70c90 into commons-app:main Jan 3, 2024
@nicolas-raoul
Copy link
Member

Thanks Rohit!

@rohit9625
Copy link
Contributor Author

Thanks to you too for guiding me :)

ShashwatKedia pushed a commit to ShashwatKedia/apps-android-commons that referenced this pull request Jan 15, 2024
* NearbyParentFragment : added referer

In file NearbyParentFragment.java, I added header property, i.e., the referer - http://maps.wikimedia.org/
and set tile source to wikimedia.

* Reworded comments

---------

Co-authored-by: Nicolas Raoul <[email protected]>
nicolas-raoul added a commit that referenced this pull request Jan 15, 2024
* Resolved issue #4513

* Localisation updates from https://translatewiki.net.

* NearbyParentFragment : added referer (#5417)

* NearbyParentFragment : added referer

In file NearbyParentFragment.java, I added header property, i.e., the referer - http://maps.wikimedia.org/
and set tile source to wikimedia.

* Reworded comments

---------

Co-authored-by: Nicolas Raoul <[email protected]>

* Localisation updates from https://translatewiki.net.

* Localisation updates from https://translatewiki.net.

* Removed butterknife from the quiz result activity (#5425)

* ConnectRefuseError: removed the proxy from properties.gradle file to allow downloading of build tools during CI. (#5434)

* NearbyParentFragment : added referer

In file NearbyParentFragment.java, I added header property, i.e., the referer - http://maps.wikimedia.org/
and set tile source to wikimedia.

* Reworded comments

* sdkmanager: added installation command for build-tools-30.0.3

* Revert "sdkmanager: added installation command for build-tools-30.0.3"

This reverts commit b3e5019.

* Update android.yml

* Update gradle.properties

* android.yml: removed extra debug commands

Removed some debug commands because they are no longer needed.

---------

Co-authored-by: Nicolas Raoul <[email protected]>

* Removed butterknife and inlined a couple of tiny methods (#5426)

* Resolves #2239 by adding a compass arrow for direction of nearest item (#5433)

* Resolves issue #2239 by adding an arrow for direction

* Removed unnecessary change in styles.xml

* spacing

* javadoc

---------

Co-authored-by: Nicolas Raoul <[email protected]>

* Added voice input for caption and description (#5415)

* Fixed Grey empty screen at Upload wizard caption step after denying files permission

* Empty commit

* Fixed loop issue

* Created docs for earlier commits

* Fixed javadoc

* Fixed spaces

* Added added basic features to OSM Maps

* Added search location feature

* Added filter to Open Street Maps

* Fixed chipGroup in Open Street Maps

* Removed mapBox code

* Removed mapBox's code

* Reformat code

* Reformatted code

* Removed rotation feature to map

* Removed rotation files and Fixed Marker click problem

* Ignored failing tests

* Added voice input feature

* Fixed test cases

* Changed caption and description text

---------

Co-authored-by: translatewiki.net <[email protected]>
Co-authored-by: Rohit Verma <[email protected]>
Co-authored-by: Nicolas Raoul <[email protected]>
Co-authored-by: Paul Hawke <[email protected]>
Co-authored-by: Kanahia <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Send Referer header from HTTP/HTTPS requests sent by the app
2 participants