Skip to content

feat(images): add openimg optimization for landing page#106

Draft
emxemirc wants to merge 11 commits intojanhesters:mainfrom
emxemirc:feat/openimg-implementation
Draft

feat(images): add openimg optimization for landing page#106
emxemirc wants to merge 11 commits intojanhesters:mainfrom
emxemirc:feat/openimg-implementation

Conversation

@emxemirc
Copy link

@emxemirc emxemirc commented Nov 16, 2025

  • Integrate openimg with native React component
  • Create /resources/images endpoint for on-demand optimization
  • Add server-side image source handler for public folder
  • Update landing hero, features, and description sections

@emirAtolye
feat(images): add openimg optimization for landing pages
0d3da0e 
  - Integrate openimg with native React component
  - Create /resources/images endpoint for on-demand optimization
  - Add server-side image source handler for public folder
  - Update landing hero, features, and description sections
@emxemirc emxemirc changed the title feat(images): add openimg optimization for landing pages feat(images): add openimg optimization for landing page Nov 16, 2025
@emirAtolye
feat(images): add security controls and tests for image optimization
a9f2e59 
  - Restrict access to public/images directory only
  - Validate file extensions (png, jpg, jpeg, webp, avif, gif)
  - Add path traversal protection
  - Check file existence before processing
  - Add comprehensive unit tests for security scenarios
@emirAtolye
fix(images): prevent path traversal and improve error caching
41527c0
@emirAtolye
test(images): add comprehensive unit and e2e tests for image optimiza…
7ec2b78 
…tion
@emirAtolye
refactor(images): use OPENIMG_CACHE_DIR environment variable
5827095
@janhesters
Copy link
Owner

janhesters commented Nov 17, 2025

Thank you so much for the contribution! Looks good already 🔥

One thing to keep in mind, this should be backwards compatible with Supabase Storage.

emirAtolye and others added 3 commits November 29, 2025 22:51
@emirAtolye
feat(images): add Supabase Storage support for avatars and logos
707547b 
  - Add getOptimizedImageUrl helper to wrap image URLs at component level
  - Extend image optimization to handle remote Supabase Storage URLs
  - Add allowlistedOrigins configuration for secure remote fetching
  - Update avatar and logo components across the app
  - Add comprehensive unit, integration and E2E tests
@emxemirc
Replace npx with bunx for commitlint
014f8af
@emxemirc
Replace npx with bunx for pre-commit checks
2002b49
@emxemirc
Copy link
Author

emxemirc commented Nov 30, 2025

Thank you so much for the contribution! Looks good already 🔥

One thing to keep in mind, this should be backwards compatible with Supabase Storage.

@janhesters
hey! I actually finished the work, but I'm having some trouble with the tests for loginAndSaveUserAccountToDatabase. When I do “change account settings + save changes,” Playwright goes back to the login screen. I haven’t had a chance to look into that issue yet. Once I check it out, I’ll move the PR from draft to ready for review.

@janhesters
Copy link
Owner

janhesters commented Dec 28, 2025

@emxemirc

hey! I actually finished the work, but I'm having some trouble with the tests for loginAndSaveUserAccountToDatabase. When I do “change account settings + save changes,” Playwright goes back to the login screen. I haven’t had a chance to look into that issue yet. Once I check it out, I’ll move the PR from draft to ready for review.

Thank you! 🔥 You might also want to rebase on main since there are a lot of conflicting changes.

Looking forward! Happy holidays 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@emxemirc @janhesters @emirAtolye