Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix 5.15 CI errors #3860

Merged
merged 4 commits into from
Dec 18, 2023
Merged

Fix 5.15 CI errors #3860

merged 4 commits into from
Dec 18, 2023

Conversation

vga91
Copy link
Collaborator

@vga91 vga91 commented Dec 18, 2023

  • added @Context URLAccessChecker urlAccessChecker where needed

Fixes due to this Core breaking-change:

  • changed http://bit.ly with https://bit.ly,
    otherwise the checkUrl throws a java.io.FileNotFoundException: http://67.199.248.10/2nXgHA2

  • added LoadS3Test.removeRegionFromUrl(url),
    since the URL with the region,
    e.g. s3://us-east-1.127.0.0.1:60082/test-bucket/test.csv?accessKey=accesskey&secretKey=secretkey,
    throws an java.net.UnknownHostException: us-east-1.127.0.0.1: nodename nor servname provided, or not known,
    and we cannot change the s3Container.putFile(<FileName>); method since is in Core

@conker84 conker84 merged commit e9f8553 into 5.15 Dec 18, 2023
1 check passed
@conker84 conker84 deleted the fix-5.15-ci-errors branch December 18, 2023 15:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants