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

NoMongo: Replace Base64 Storage with MinIO Client in OrgPostCard.tsx #3721

Open
NishantSinghhhhh opened this issue Feb 22, 2025 · 3 comments
Assignees

Comments

@NishantSinghhhhh
Copy link
Contributor

NishantSinghhhhh commented Feb 22, 2025

Problem
OrgPostCard.tsx currently uses Base64 encoding for storing images, leading to increased memory usage and slower load times.

Solution
Use MinIO client to upload images and store only file URLs, optimizing performance and reducing payload size.

Alternatives Considered

  • Keeping Base64 (inefficient for large files).
  • Using third-party storage services (MinIO provides a self-hosted alternative).

Approach

  1. Remove Base64 encoding.
  2. Implement MinIO file uploads and store URLs in the database.
  3. Ensure proper error handling and security measures.

Additional Context
This update will improve performance, scalability, and user experience for handling post images.


NOTE

  1. We recently discovered a flaw in the code base where there are some XSS and CORS vulnerabilities. This occurs when the API and Admin servers run on different machines.

  2. You will notice this if you configure your Admin app on your local machine to use the API running on https://test.talawa.io/graphql.

  3. The errors in your browser will look like this:

    Image

  4. As part of this issue you will need to ensure that the browser only interacts with the API through the Admin server and never with the API direclty.

@Akshat2gupta
Copy link

Hi @NishantSinghhhhh, I’d love to work on this issue! Since optimizing image storage with MinIO is a great improvement for performance and security, I’d be happy to implement the changes.

Could you please assign this issue to me? Thanks!

@palisadoes palisadoes changed the title Feature Request : Replace Base64 Storage with MinIO Client in OrgPostCard.tsx NoMongo: Replace Base64 Storage with MinIO Client in OrgPostCard.tsx Feb 24, 2025
@palisadoes
Copy link
Contributor

PTAL at the XSS and CORS vulnerabilities mentioned above that you will need to address as part of this.

Akshat2gupta pushed a commit to Akshat2gupta/talawa-admin that referenced this issue Mar 9, 2025
@palisadoes
Copy link
Contributor

How's progress on this? It needs to be completed as a GSoC prerequisite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Backlog
Development

No branches or pull requests

3 participants