Skip to content

Integration of weather support for sessions#41

Open
PrankN wants to merge 3 commits intoRedback-Operations:mainfrom
PrankN:main
Open

Integration of weather support for sessions#41
PrankN wants to merge 3 commits intoRedback-Operations:mainfrom
PrankN:main

Conversation

@PrankN
Copy link

@PrankN PrankN commented Sep 6, 2025

Summary

This PR introduces weather support for sessions

Key Features

  • Weather API Endpoints:
    • GET /api/sessions/<session_id>/weather: fetch weather data aligned with session time/location.
    • Internal logic uses lat/lon and timestamp to retrieve external weather (or mock/local fallback).
  • CORS Support for Frontend:
    • CORS headers added to allow requests from http://localhost:5173
    • Required for local development of the React/Vite frontend.

Notes

  • No breaking changes to existing endpoints.
  • Weather source currently mocked (or weather.py can be extended).
  • Endpoints will return [] or safe fallback if no data is found.
  • Confirmed CORS headers applied and sessions fetchable by frontend app.

@github-actions
Copy link

github-actions bot commented Sep 6, 2025

🔒 Security Scan Results

🔒 Security Scan Results
=========================

Bandit Scan Results:
-------------------
Run started:2025-09-06 04:06:24.190002

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1198
	Total lines skipped (#nosec): 0
	Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
	Total issues (by severity):
		Undefined: 0
		Low: 14
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 14
Files skipped (2):
	./api/sessions.py (syntax error while parsing AST from file)
	./app.py (syntax error while parsing AST from file)

Dependency Check Results:
-----------------------

No critical security issues detected.

The code has passed all critical security checks.

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.

1 participant