Skip to content

Fix verify_token import path in authentication dependencies#83

Open
ZainabTravadi wants to merge 1 commit into
devloperdevesh:mainfrom
ZainabTravadi:fix/dependencies-import-path
Open

Fix verify_token import path in authentication dependencies#83
ZainabTravadi wants to merge 1 commit into
devloperdevesh:mainfrom
ZainabTravadi:fix/dependencies-import-path

Conversation

@ZainabTravadi

Copy link
Copy Markdown

Description

This PR fixes an incorrect import path in app/core/dependencies.py.

The module was importing verify_token from core.security, but the implementation exists under app.core.security. This could lead to a ModuleNotFoundError when importing authentication dependencies in a clean environment.

The fix updates the import to use the correct package path and adds a regression test to ensure the dependency module imports successfully.

Type of Change

  • Bug Fix
  • New Feature
  • Documentation
  • UI Improvement
  • Refactor

Related Issue

Closes #82

Checklist

  • Code tested locally
  • No unnecessary files added
  • Documentation updated

Notes

Changes

  • Updated verify_token import from core.security to app.core.security.
  • Added a regression test to verify app.core.dependencies imports correctly and resolves verify_token from the expected module.

Impact

  • Prevents import-time failures caused by an invalid module path.
  • Does not modify authentication logic or runtime behavior beyond fixing the import resolution.

This contribution is part of GSSoC 2026.

@vercel

vercel Bot commented May 30, 2026

Copy link
Copy Markdown

@ZainabTravadi is attempting to deploy a commit to the devloperdevesh's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Karanjot786

Copy link
Copy Markdown

Hey @ZainabTravadi! Saw your work on GSSoC 2026.

We are building TermUI, a TypeScript terminal UI framework with React-style hooks and JSX, rendered entirely in the terminal.

We have 99 unassigned GSSoC issues open, including well-scoped bug fixes with clear repro steps. Your TypeScript background transfers directly.

Karanjot, TermUI maintainer

@ZainabTravadi

Copy link
Copy Markdown
Author

Hi @Karanjot786

Thank you for reaching out! TermUI looks really interesting, especially the React-style hooks and JSX approach for terminal applications.

I’m currently exploring a few GSSoC repositories focused on backend systems, AI infrastructure, and developer tooling, but I’d be happy to take a look at TermUI’s open issues as well. I’ll review the repository and contribution guidelines and see where I can contribute.

Looking forward to exploring the project!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

app.core.dependencies imports verify_token from non-existent core.security module

2 participants