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

Bytes Length is confusing #93

Closed
blankdots opened this issue Feb 18, 2019 · 3 comments
Closed

Bytes Length is confusing #93

blankdots opened this issue Feb 18, 2019 · 3 comments
Assignees
Labels
Dataedge Issue is related to Dataedge microservice Stable release

Comments

@blankdots
Copy link
Contributor

blankdots commented Feb 18, 2019

Description

When retrieving the file from DataEdge the implementation extracts 16 bytes https://github.com/EGA-archive/ega-data-api/blob/master/ega-data-api-dataedge/src/main/java/eu/elixir/ega/ebi/dataedge/service/internal/RemoteFileServiceImpl.java#L782, a crypt4gh needs to extract 32 bytes as shown here: https://github.com/EGA-archive/ega-data-api/blob/test/m4/ega-data-api-dataedge/src/main/java/eu/elixir/ega/ebi/dataedge/service/internal/RemoteFileServiceImpl.java#L782

Then there are bytes removed again here: https://github.com/EGA-archive/ega-data-api/blob/test/m4/ega-data-api-dataedge/src/main/java/eu/elixir/ega/ebi/dataedge/service/internal/RemoteFileServiceImpl.java#L159

Then there is this prefix: https://github.com/EGA-archive/ega-data-api/blob/test/m4/ega-data-api-dataedge/src/main/java/eu/elixir/ega/ebi/dataedge/service/internal/RemoteFileServiceImpl.java#L774 of 16 bytes.

The explanation is here: https://github.com/EGA-archive/ega-data-api/blob/test/m4/ega-data-api-dataedge/src/main/java/eu/elixir/ega/ebi/dataedge/service/internal/RemoteFileServiceImpl.java#L188-L190 ?

The implementation is a confusing and could be better in the way it handles distributing plain and encrypted files.

Definition of Done

Figure out a flexible solution in dealing with the file distribution via DataEdge.

How to test

Unit test and integration tests for different file formats, encrypted or not with different algorithms.

@blankdots blankdots added enhancement New feature or request Dataedge Issue is related to Dataedge microservice labels Feb 18, 2019
@juhtornr
Copy link
Collaborator

@anandmohan777 can replicate this and will provide details

@juhtornr
Copy link
Collaborator

and @omllobet as well.

@juhtornr juhtornr removed the enhancement New feature or request label Apr 17, 2019
@anandmohan777 anandmohan777 reopened this May 3, 2019
@blankdots
Copy link
Contributor Author

Let us finish this as well: #131 and we can close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dataedge Issue is related to Dataedge microservice Stable release
Projects
None yet
Development

No branches or pull requests

4 participants