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

API v1 & major refactoring: Hono, file read optimizations, and upload streaming #16

Draft
wants to merge 185 commits into
base: main
Choose a base branch
from

Conversation

linkability
Copy link
Contributor

@linkability linkability commented Oct 4, 2023

#15

A better & more sensible API which is actually usable for developers

@linkability linkability self-assigned this Oct 4, 2023
@linkability linkability marked this pull request as draft October 4, 2023 00:43
@nbitzz nbitzz added the enhancement New feature or request label Oct 4, 2023
@nbitzz nbitzz added this to the 1.4 milestone Oct 4, 2023
@nbitzz nbitzz linked an issue Oct 4, 2023 that may be closed by this pull request
@nbitzz nbitzz mentioned this pull request Oct 4, 2023
9 tasks
@Jack5079
Copy link
Member

Jack5079 commented Oct 5, 2023

A mollermeth of sorts

@nbitzz
Copy link
Collaborator

nbitzz commented Oct 5, 2023

true!

Copy link
Collaborator

@nbitzz nbitzz left a comment

Choose a reason for hiding this comment

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

make DELETE /account/me ignore body & just delete the account, from now on we'll have the client implement file deletion instead

@Jack5079
Copy link
Member

Jack5079 commented Oct 6, 2023

Genius! Mr. 2283 and Mr. Impersonator will never find out

@nbitzz nbitzz requested a review from Jack5079 June 18, 2024 18:19
@nbitzz
Copy link
Collaborator

nbitzz commented Jun 18, 2024

i think all that's left now is maybe a little bit of cleanup and implementing account recovery but aside from that it's done

<body>
<h1>monofile password reset</h1>
<form>
<input name="password" type="password">
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<input name="password" type="password">
<input name="password" type="password" autocomplete="new-password">

https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/autocomplete#sect13

A new password. When creating a new account or changing passwords, this should be used for an "Enter your new password" or "Confirm new password" field, as opposed to a general "Enter your current password" field that might be present. This may be used by the browser both to avoid accidentally filling in an existing password and to offer assistance in creating a secure password.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-v1 Stuff related to & to be implemented in api-v1 enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

make an actual good api Optimize, stabilize file downloads Refactor API v0
7 participants