Skip to content

Block redirect in IP2Geo and move validation to transport action#782

Merged
heemin32 merged 1 commit intoopensearch-project:mainfrom
heemin32:3.2.0-dev
Aug 13, 2025
Merged

Block redirect in IP2Geo and move validation to transport action#782
heemin32 merged 1 commit intoopensearch-project:mainfrom
heemin32:3.2.0-dev

Conversation

@heemin32
Copy link
Copy Markdown
Collaborator

Description

  1. Block redirect request
  2. Move manifest validation check inside transport action

Related Issues

N/A

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@heemin32 heemin32 changed the title Changes on IP2Geo processors Block redirect in IP2Geo and move validation to transport action Aug 12, 2025
1. Block redirect request
2. Move manifest validation check inside transport action

Signed-off-by: Heemin Kim <heemin@amazon.com>
@heemin32
Copy link
Copy Markdown
Collaborator Author

heemin32 commented Aug 13, 2025

The failure is due to redirection used in manifest.json file. I updated it so it will pass once this PR is merged.
https://github.com/opensearch-project/geospatial/pull/782/files#diff-5d5fe15bc508f78e573b250f748d91a0d680b0a4ba0e55f7e30e96d9de06f00aL2

For changelog, I updated in release note directly.

@heemin32 heemin32 merged commit bbac6cb into opensearch-project:main Aug 13, 2025
11 of 13 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 13, 2025
1. Block redirect request
2. Move manifest validation check inside transport action

Signed-off-by: Heemin Kim <heemin@amazon.com>
(cherry picked from commit bbac6cb)
heemin32 added a commit that referenced this pull request Aug 13, 2025
1. Block redirect request
2. Move manifest validation check inside transport action


(cherry picked from commit bbac6cb)

Signed-off-by: Heemin Kim <heemin@amazon.com>
Co-authored-by: Heemin Kim <heemin@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants