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

S3support #293

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

S3support #293

wants to merge 7 commits into from

Conversation

rjaycocks
Copy link

Added support for S3.
Handle open headers, will then retrieve the data from the footer (which should be closed (this is checked)).

src\main\java\com\netflix\imflibrary\st0377\HeaderPartition.java:1210: error: unmappable character for encoding Cp1252
        L Γå? Empty list that will contain the sorted nodes
…allows the header to be open, in which case the meta data should then be obtained from the closed footer partition (which is usually the last partition in the file, referenced by the last entry in the RIP).
…tain the last closed footer from the RIP. Warn if the header is not complete.
…ader is not complete

Fixed handling of the partition offsets, need to add the start of the RIP to the offsets when obtaining the last footer partition.
Added maven nexus publishing using Profiles
…. Assume that the http style paths are URL Encoded, whilst the S3 style paths are not URL encoded. This is based on the links in the AWS S3 console.

Fixed processing of the us-east-1 region for buckets. When trying to determine the bucket location, the AWS sdk returns a region called US which needs to be converted to us-east-1.
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.

1 participant