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

customer/uk-it-7 #489

Merged
merged 20 commits into from
Jan 3, 2024
Merged

customer/uk-it-7 #489

merged 20 commits into from
Jan 3, 2024

Conversation

milanmajchrak
Copy link
Collaborator

Phases MP MM MB MR JM Total
ETA 0 0 0 0 0 0
Developing 0 0 0 0 0 0
Review 0 0 0 0 0 0
Total - - - - - 0
ETA est. 0
ETA cust. - - - - - 0

Problem description

milanmajchrak and others added 20 commits November 15, 2023 12:29
* The admin could download restricted bitstreams.

* The user cannot download bitstream if he/she is not signed in.

* Cannot obtain context with user.

* The user is already fetched from the context.

* Added docs.

* Do not use endpoint for downloading the single file because it already exists - remove it.

* Fixed AuthorizationRestControllerIT tests.
* Find bitstream format using mimetype not ID.

* Updated tests according to fix.
* Updated `handle.canonical.prefix` to hdl handle.

* Integration test cannot have changed handle.canonical.prefix.
* The user registration is added when the eperson is created by the ui.

* try using newer checkout, same as upstream, since error is with git

---------

Co-authored-by: MajoBerger <[email protected]>
* Add logging of downloading restricted bitstreams.

* Log downloading of every bitstream not only restricted ones.

* Added docs

* Refactored logging.
* givenname and last name values are converted to UTF-8 encoding

* Delete eperson in tests
Fixed emails sending - added arguments and cfg properties that are sent in the email. (#470)
* DSpace#8585 Add submitter information to provenance metadata

* Cherry picked fix from vanilla and added test to check if the Item provenance metadata is added

---------

Co-authored-by: Adán Román Ruiz <[email protected]>
* Created endpoint for fetching IP address with tests.

* Made the code more understable
* fixes ufal#1096 (#471)

* Added publisher filter in the integration test check

---------

Co-authored-by: Ondřej Košarko <[email protected]>
* Made previewing of the file configurable.

* The default value of the previewing the file must be true in the integration tests.
Exposed max file limit for upload from the cfg. (#477)
* Fixed comments in the clarin emails (#484)
* Added cfg property to enable/disable showing of the page with attributes passed from the idp.
* The bitstream data are stored in to local store after uploading to S3

* The bitstream is removed from the S3 and local assetstore

* Store number is specific if all storages are synchronized.

* Return store number in the BitstreamRest object

* Find out which store is used - it could be synchronized stores number.

* Constant is moved to the Bitstream class

* Synchronization of storages is not allowed by default - set up it in the test environment.

* Added docs

* Removed constant from the Bitstream class - it wasn't consistent

* Overriden BitstreamStorageServiceImpl by custom SyncBitstreamStorageServiceImpl

* Removed ClarinS3BitStoreService.java to SyncS3BitStoreService

* Added doc and refactoring.
* Fixed UserCheck - There was created a wrong select

* Fixed ChecksumCheck - the session was closed but there were some requests do the database then
* Load version from the specific file and show content in the root endpoint.

* Fixed checkstyle issue

* DO not log anything if the VERSION file does not exist. Added VERSION file path into cfg property.

* Append every line into final String.
* Cherry picked fix S3 optimalization fixes from Vanilla

* Cherry picked adding of pagination on cleanup

* Updated Sync services according to S3 optimization fix

* The checksum value is compared between S3 and local assetstore and the new result is inserted to the database.

* Added docs and refactoring

* The checksum values are exposed by REST API

* Fixed failing tests

---------

Co-authored-by: Tim Donohue <[email protected]>
Co-authored-by: Luca Giamminonni <[email protected]>
@milanmajchrak milanmajchrak changed the title Customer/uk it 7 customer/uk-it-7 Jan 3, 2024
@milanmajchrak milanmajchrak self-assigned this Jan 3, 2024
@milanmajchrak milanmajchrak merged commit 35645f2 into customer/uk Jan 3, 2024
7 checks passed
@MajoBerger MajoBerger deleted the customer/uk-it-7 branch August 5, 2024 10:06
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.

2 participants