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

feat: FileBone(public=True) for public files #1241

Merged
merged 39 commits into from
Sep 23, 2024
Merged

Conversation

akelch
Copy link
Member

@akelch akelch commented Aug 20, 2024

This PR adds public files. Public files are stored in an extra bucket that follows this name: public-dot-{__PROJECT_ID}
The bucket must be created with ACL rights.

With getUploadUrl, the parameter public is used to decide whether it is a public file or not.
Filebones with flag public=True can only refer to a public file

Public files again deliver a servingurl.

@akelch akelch added the viur-meeting Issues to discuss in the next ViUR meeting label Aug 20, 2024
@phorward phorward changed the title Allow Public files feat: FileBone(public=True) for public files without signature authentication Aug 20, 2024
@phorward phorward changed the title feat: FileBone(public=True) for public files without signature authentication feat: FileBone(public=True) for public files Aug 20, 2024
@phorward phorward added this to the ViUR-core v3.6 milestone Aug 20, 2024
@phorward phorward added feature New feature or request main labels Aug 20, 2024
@phorward phorward changed the base branch from main to develop August 27, 2024 16:49
Copy link
Member

@phorward phorward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@akelch can you test this branch again with my refactoring?
@sveneberth @ArneGudermann looking phorward for your review!

src/viur/core/modules/file.py Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Outdated Show resolved Hide resolved
src/viur/core/modules/file.py Show resolved Hide resolved
src/viur/core/modules/file.py Show resolved Hide resolved
Co-authored-by: Sven Eberth <se@mausbrand.de>
phorward
phorward previously approved these changes Sep 20, 2024
Copy link
Member

@phorward phorward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tested together with @akelch.

ArneGudermann
ArneGudermann previously approved these changes Sep 23, 2024
@phorward phorward added Priority: High After critical issues are fixed, these should be dealt with before any further issues. viur-meeting Issues to discuss in the next ViUR meeting labels Sep 23, 2024
@phorward phorward dismissed stale reviews from ArneGudermann and themself via a94b922 September 23, 2024 20:15
@phorward phorward merged commit ccde93c into develop Sep 23, 2024
8 checks passed
@phorward phorward deleted the feat/public-files branch September 23, 2024 21:15
phorward added a commit to phorward/viur-core that referenced this pull request Sep 24, 2024
phorward added a commit that referenced this pull request Sep 30, 2024
This is a follow-up on #1241

---------

Co-authored-by: Sven Eberth <mail@sveneberth.de>
@phorward phorward removed the viur-meeting Issues to discuss in the next ViUR meeting label Oct 11, 2024
phorward added a commit that referenced this pull request Dec 6, 2024
just add servingurl to default refkeys in filebone.
Enhancement for #1241.

---------

Co-authored-by: Jan Max Meyer <jmm@phorward.de>
phorward added a commit to phorward/viur-core that referenced this pull request Dec 11, 2024
…ork#1344

Added required `RecordBone.refresh` for Fluidpages.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request Priority: High After critical issues are fixed, these should be dealt with before any further issues.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

4 participants