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

Document revamp & server-side encryption #86

Merged
merged 7 commits into from
Apr 14, 2024
Merged

Conversation

inetol
Copy link
Contributor

@inetol inetol commented Apr 13, 2024

Este PR cambia la estructura de los documentos para poder reducir el overhead al descomprimir los mismos, se hace una mejor implementación de la caducidad de los documentos para que sean configurables globalmente (inicial #83), se añade la posibilidad de que el servidor encripte los documentos al proporcionar una contraseña y más bugfixes

@inetol inetol self-assigned this Apr 13, 2024
@inetol inetol linked an issue Apr 13, 2024 that may be closed by this pull request
@inetol
Copy link
Contributor Author

inetol commented Apr 13, 2024

@tnfAngel Podrías mirar a ver como solucionar los 2 errores de lint en AccessV2 y PublishV2?

@inetol inetol marked this pull request as ready for review April 13, 2024 20:21
@tnfAngel
Copy link
Member

@tnfAngel Podrías mirar a ver como solucionar los 2 errores de lint en AccessV2 y PublishV2?

vale

@tnfAngel
Copy link
Member

tnfAngel commented Apr 13, 2024

bien, el error se debe a que el access/publish del DocumentHandler devuelve una union entre las dos versiones en vez de devolver la respuestas especifica usando la version del imput, por lo que he hecho eso

hay un fix mas cutre que seria simplemente hacer las propiedades nuevas de v2 todas opcionales pero no conviene, de momento lo voy a arreglar usando genericos y tipos condicionales (lo del throw nunca se ejecutará pero si no lo pongo TS lloraba)

(los tipos de respuesta se pueden dejar asi de momento, quizas haya otra solución más elegante)

@inetol
Copy link
Contributor Author

inetol commented Apr 14, 2024

Al final creo que voy a poner la logica en cada version de endpoint porque a la minima que se cambie algo y TS no sepa que se esta haciendo tocara hacer chanchullos

@inetol inetol merged commit 7cc4d90 into dev Apr 14, 2024
3 checks passed
@inetol inetol deleted the revamp-document-structure branch April 14, 2024 07:25
@Mrgaton
Copy link
Member

Mrgaton commented Apr 14, 2024

Hay que encontrar una solucion al raw

@tnfAngel
Copy link
Member

hmm se me ocurrió algo para al menos no tener que tipar las responses

@inetol inetol mentioned this pull request Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cifrar documentos protegidos por contraseña con la propia contraseña
3 participants