-
-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mdk/add client tests #67
base: dev
Are you sure you want to change the base?
Conversation
* add error handling when removing iptables rule (WLAN-Pi#51)
# Conflicts: # wlanpi_core/services/network_service.py # wlanpi_core/utils/general.py # wlanpi_core/utils/network.py
and some other cleanups and fixes
…e is used. added a "current network" endpoint
…n via interface name
return Response(content=ve.error_msg, status_code=ve.status_code) | ||
except Exception as ex: | ||
log.error(ex) | ||
return Response(content=f"Internal Server Error: {ex}", status_code=500) |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
return Response(content=ve.error_msg, status_code=ve.status_code) | ||
except Exception as ex: | ||
log.error(ex, exc_info=ex) | ||
return Response(content=f"Internal Server Error: {ex}", status_code=500) |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
return Response(content=ve.error_msg, status_code=ve.status_code) | ||
except Exception as ex: | ||
log.error(ex, exc_info=ex) | ||
return Response(content=f"Internal Server Error: {ex}", status_code=500) |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
return Response(content=ve.error_msg, status_code=ve.status_code) | ||
except Exception as ex: | ||
log.error(ex, exc_info=ex) | ||
return Response(content=f"Internal Server Error: {ex}", status_code=500) |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
return Response(content=ve.error_msg, status_code=ve.status_code) | ||
except Exception as ex: | ||
log.error(ex, exc_info=ex) | ||
return Response(content=f"Internal Server Error: {ex}", status_code=500) |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
return Response(content=ve.error_msg, status_code=ve.status_code) | ||
except Exception as ex: | ||
log.error(ex, exc_info=ex) | ||
return Response(content=f"Internal Server Error: {ex}", status_code=500) |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Type of change
This is a draft PR built against my existing WiFi overhaul branch. Issue and cleanup to come, just want it to be visible.
Pick at least one.
Breaking change
Does this Pull Request introduce a breaking change?
No known breaks
Proposed change
Adds various tests and tools leveraging the new WLAN capabilities. These include, all scopable to any interface:
Not wlan specific:
Testing
Please explain how you tested this change manually, and if applicable, what new tests you added.
No new automated tests (yet)
No affects on existing tests
Checklist
Related Issues/PRs
Pick at least one.