[PR #309] [CLOSED] Remove authentication for personal use #280

Closed
opened 2026-03-02 23:34:56 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AJaySi/ALwrity/pull/309
Author: @KarmSakha
Created: 11/16/2025
Status: Closed

Base: mainHead: claude/remove-authentication-01PjY5jXrmPnbqRNBXakktDY


📝 Commits (1)

  • 6faf8f3 Remove authentication for personal use

📊 Changes

3 files changed (+70 additions, -715 deletions)

View changed files

📝 backend/middleware/auth_middleware.py (+24 -220)
📝 frontend/src/App.tsx (+43 -300)
📝 frontend/src/api/client.ts (+3 -195)

📄 Description

  • Backend: Modified auth middleware to always return static user (personal_user)
  • Backend: Simplified auth functions to bypass Clerk authentication
  • Frontend: Removed ClerkProvider and all Clerk dependencies from App.tsx
  • Frontend: Removed ProtectedRoute components from all routes
  • Frontend: Simplified routing - removed authentication checks
  • Frontend: Removed auth token injection from API client interceptors
  • Frontend: Set static user_id in localStorage for personal use

The application now runs without authentication, suitable for personal/single-user use.

Pull Request

📝 Description

Brief description of changes made in this PR.

🔄 Type of Change

  • 🐛 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 not work as expected)
  • 📚 Documentation update
  • 🎨 Style/UI changes
  • ♻️ Code refactoring
  • Performance improvements
  • 🧪 Test additions/updates

Closes #(issue number)
Fixes #(issue number)
Related to #(issue number)

🧪 Testing

  • Backend tests pass
  • Frontend tests pass
  • Manual testing completed
  • Cross-browser testing (if applicable)
  • Mobile testing (if applicable)

📸 Screenshots (if applicable)

Add screenshots to help explain your changes.

Before

After

🏷️ Component/Feature

Which component or feature is affected?

  • Blog Writer
  • SEO Dashboard
  • Content Planning
  • Facebook Writer
  • LinkedIn Writer
  • Onboarding
  • Authentication
  • API
  • Database
  • GSC Integration
  • Subscription System
  • Monitoring/Billing
  • Documentation
  • Other: _______________

📋 Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

ALwrity-Specific Checklist

  • API endpoints follow RESTful conventions
  • AI service integrations handle rate limits and errors gracefully
  • Content generation includes proper validation and sanitization
  • Database migrations are included if schema changes are made
  • Environment variables are documented in env_template.txt
  • Security considerations have been addressed
  • Performance impact has been considered
  • User experience is consistent with existing features

🔍 Code Quality

  • Code is properly formatted
  • No console.log statements left in production code
  • Error handling is implemented where needed
  • Performance considerations have been addressed
  • Security considerations have been addressed

📚 Documentation

  • README updated (if needed)
  • API documentation updated (if needed)
  • Code comments added for complex logic
  • Changelog updated (if applicable)

🚀 Deployment Notes

Any special deployment considerations or environment variables needed.

🔗 Additional Context

Add any other context about the pull request here.

👥 Reviewers

Tag specific reviewers if needed:
@AJaySi @uniqueumesh @DikshaDisciplines


Thank you for contributing to ALwrity! 🎉


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/AJaySi/ALwrity/pull/309 **Author:** [@KarmSakha](https://github.com/KarmSakha) **Created:** 11/16/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `claude/remove-authentication-01PjY5jXrmPnbqRNBXakktDY` --- ### 📝 Commits (1) - [`6faf8f3`](https://github.com/AJaySi/ALwrity/commit/6faf8f3a6b48d376e1799ead5c134a3fb3a0a6ba) Remove authentication for personal use ### 📊 Changes **3 files changed** (+70 additions, -715 deletions) <details> <summary>View changed files</summary> 📝 `backend/middleware/auth_middleware.py` (+24 -220) 📝 `frontend/src/App.tsx` (+43 -300) 📝 `frontend/src/api/client.ts` (+3 -195) </details> ### 📄 Description - Backend: Modified auth middleware to always return static user (personal_user) - Backend: Simplified auth functions to bypass Clerk authentication - Frontend: Removed ClerkProvider and all Clerk dependencies from App.tsx - Frontend: Removed ProtectedRoute components from all routes - Frontend: Simplified routing - removed authentication checks - Frontend: Removed auth token injection from API client interceptors - Frontend: Set static user_id in localStorage for personal use The application now runs without authentication, suitable for personal/single-user use. # Pull Request ## 📝 Description Brief description of changes made in this PR. ## 🔄 Type of Change - [ ] 🐛 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 not work as expected) - [ ] 📚 Documentation update - [ ] 🎨 Style/UI changes - [ ] ♻️ Code refactoring - [ ] ⚡ Performance improvements - [ ] 🧪 Test additions/updates ## 🎯 Related Issues Closes #(issue number) Fixes #(issue number) Related to #(issue number) ## 🧪 Testing - [ ] Backend tests pass - [ ] Frontend tests pass - [ ] Manual testing completed - [ ] Cross-browser testing (if applicable) - [ ] Mobile testing (if applicable) ## 📸 Screenshots (if applicable) Add screenshots to help explain your changes. ### Before <!-- Add before screenshots here --> ### After <!-- Add after screenshots here --> ## 🏷️ Component/Feature Which component or feature is affected? - [ ] Blog Writer - [ ] SEO Dashboard - [ ] Content Planning - [ ] Facebook Writer - [ ] LinkedIn Writer - [ ] Onboarding - [ ] Authentication - [ ] API - [ ] Database - [ ] GSC Integration - [ ] Subscription System - [ ] Monitoring/Billing - [ ] Documentation - [ ] Other: _______________ ## 📋 Checklist - [ ] My code follows the project's style guidelines - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged and published ### ALwrity-Specific Checklist - [ ] API endpoints follow RESTful conventions - [ ] AI service integrations handle rate limits and errors gracefully - [ ] Content generation includes proper validation and sanitization - [ ] Database migrations are included if schema changes are made - [ ] Environment variables are documented in env_template.txt - [ ] Security considerations have been addressed - [ ] Performance impact has been considered - [ ] User experience is consistent with existing features ## 🔍 Code Quality - [ ] Code is properly formatted - [ ] No console.log statements left in production code - [ ] Error handling is implemented where needed - [ ] Performance considerations have been addressed - [ ] Security considerations have been addressed ## 📚 Documentation - [ ] README updated (if needed) - [ ] API documentation updated (if needed) - [ ] Code comments added for complex logic - [ ] Changelog updated (if applicable) ## 🚀 Deployment Notes Any special deployment considerations or environment variables needed. ## 🔗 Additional Context Add any other context about the pull request here. ## 👥 Reviewers Tag specific reviewers if needed: @AJaySi @uniqueumesh @DikshaDisciplines --- **Thank you for contributing to ALwrity!** 🎉 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-02 23:34:56 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ALwrity#280
No description provided.