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

fix(server): upload limit #348

Closed
wants to merge 1 commit into from
Closed

Conversation

tessus
Copy link
Collaborator

@tessus tessus commented Sep 27, 2024

Description

Update code to limit the file upload.

Motivation and Context

see #336
and https://stackoverflow.com/a/78399675

How Has This Been Tested?

It does not seem to work.

Changelog Entry

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (no code change)
  • Refactor (refactoring production code)
  • Other

Checklist:

Will be done after moving it out of draft status.

  • My code follows the code style of this project.
  • I have updated the documentation accordingly.
  • I have formatted the code with rustfmt.
  • I checked the lints with clippy.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@tessus
Copy link
Collaborator Author

tessus commented Sep 27, 2024

oops. sorry. This was the wrong code. I tried different ways. This one does not compile.

P.S.: now I remember why I didn't create the draft. I screwed up my dev box and have to find the correct code or write it again.

@tessus tessus closed this Sep 27, 2024
@orhun
Copy link
Owner

orhun commented Sep 27, 2024

oops. sorry. This was the wrong code. I tried different ways. This one does not compile.

np :)

P.S.: now I remember why I didn't create the draft. I screwed up my dev box and have to find the correct code or write it again.

damn :/ best of luck with that

@tessus tessus deleted the fix/upload-limit branch October 25, 2024 01:14
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