Skip to content

Feature/service logging#68

Merged
prske merged 7 commits intoagstack:mainfrom
EmmanuelNiyonshuti:feature/service-logging
Feb 20, 2026
Merged

Feature/service logging#68
prske merged 7 commits intoagstack:mainfrom
EmmanuelNiyonshuti:feature/service-logging

Conversation

@EmmanuelNiyonshuti
Copy link
Copy Markdown
Contributor

Add logging

Any related issues

#6

What this does

  • Adds logging configuration in logging_config.py
  • Sets up logging on application startup via lifespan
  • Adds logging to login, logout, and registration endpoints
  • Log level can be configured via LOG_LEVEL env variable (default to INFO)

Why it's needed

Makes it easier to debug the service when it's running in production

Example logs

Successful login:

irrigation-1  | 2026-01-21T13:32:34.961Z INFO     [api.api_v1.endpoints.login:26] - Login attempt for user: user@example.com
irrigation-1  | 2026-01-21T13:32:35.047Z INFO     [api.api_v1.endpoints.login:43] - Successful login for user: user@example.com

Failed login:

irrigation-1  | 2026-01-21T13:33:59.734Z INFO     [api.api_v1.endpoints.login:26] - Login attempt for user:  user@example.com
irrigation-1  | 2026-01-21T13:33:59.737Z WARNING  [api.api_v1.endpoints.login:36] - Failed login attempt - Invalid credentials for user:  user@example.com

@prske
Copy link
Copy Markdown
Collaborator

prske commented Jan 22, 2026

Hi @EmmanuelNiyonshuti

Thank you for contributing to this project.
This seems very nice.

Currently, we are working on two features for this repository, once we finish them, we will come back to this PR.

Regards!

@prske
Copy link
Copy Markdown
Collaborator

prske commented Feb 11, 2026

Hi @EmmanuelNiyonshuti

We have updated IRM service and now there are some merge conflicts...

Can you please check them out and renew this PR?

Thanks and regards.

@EmmanuelNiyonshuti
Copy link
Copy Markdown
Contributor Author

EmmanuelNiyonshuti commented Feb 11, 2026

Hi @EmmanuelNiyonshuti

We have updated IRM service and now there are some merge conflicts...

Can you please check them out and renew this PR?

Thanks and regards.

Hi @prske
Thank you for letting me know. I have checked the branch and resolved the merge conflict

@prske
Copy link
Copy Markdown
Collaborator

prske commented Feb 18, 2026

Hi @EmmanuelNiyonshuti

Thanks for fixing the conflicts.

I'm also adding my college as a reviewer since he has done most of the development in this service.
As far as I'm concern, this looks very nice.

Regards.

@EmmanuelNiyonshuti
Copy link
Copy Markdown
Contributor Author

Hi @EmmanuelNiyonshuti

Thanks for fixing the conflicts.

I'm also adding my college as a reviewer since he has done most of the development in this service. As far as I'm concern, this looks very nice.

Regards.

You're welcome!

Copy link
Copy Markdown
Collaborator

@vlf-stefan-drobic vlf-stefan-drobic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@prske prske merged commit bda6026 into agstack:main Feb 20, 2026
@prske prske mentioned this pull request Feb 20, 2026
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.

3 participants