Skip to content

🚚 release #107

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 30 commits into from
May 16, 2025
Merged

🚚 release #107

merged 30 commits into from
May 16, 2025

Conversation

@ltucker
Copy link
Contributor

@ltucker ltucker commented May 16, 2025

  • adds a user interface for managing diode client credentials
  • adds addional plugin configuration for authenticating calls to diode from netbox plugin

arthanson and others added 30 commits May 9, 2025 12:15
@arthanson
OBS-1046 add base for Client Credentials
69d0ae7
@arthanson
OBS-1046 add base for Client Credentials
2a0bc62
@arthanson
OBS-1046 add base for Client Credentials
f1fb759
@ltucker
feat: add api client for diode auth credentials api
c427d24
@arthanson
feat: add api client for diode auth credentials api (#104)
537f45a
@arthanson
OBS-1046 add base for Client Credentials
6c1a63d
@ltucker
fix: post json
5cc3470
@ltucker
fix: fixes to client api
a6ece58
@arthanson
OBS-1046 display token after add
bf9c3ac
@ltucker
add additional detail to README about diode credential
2f5186a
@arthanson
OBS-1046 delete
2279a97
@arthanson
Merge branch 'OBS-1046-client' of github.com:netboxlabs/diode-netbox-…
c08aec7 
…plugin into OBS-1046-client
@ltucker
fix: no response from delete, don't parse
63ae897
@arthanson
OBS-1046 ui fixes
205fb7a
@arthanson
OBS-1046 ui fixes
2e94b29
@arthanson
OBS-1046 ui fixes
9f88f6c
@arthanson
OBS-1046 fix secret copy
b3301c5
@arthanson
OBS-1046 fix delete cancel
d86815f
@ltucker
fix: validate client id before requesting
69e60a1
@ltucker
add unit tests for diode clients api wrapper
1724ae6
@ltucker @github-advanced-security
Potential fix for code scanning alert no. 23: Log Injection
ce7e982 
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@ltucker @github-advanced-security
Potential fix for code scanning alert no. 25: Log Injection
d309a0c 
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@ltucker @github-advanced-security
Potential fix for code scanning alert no. 24: Log Injection
9656e9b 
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@ltucker @github-advanced-security
Potential fix for code scanning alert no. 22: URL redirection from re…
e6e1731 
…mote source

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@ltucker @github-advanced-security
Potential fix for code scanning alert no. 26: Log Injection
9e0af56 
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@ltucker
linting
4df7063
@ltucker
display specific message if no diode credential is configured
fee9324
@ltucker @mfiedorowicz
Update netbox_diode_plugin/views.py
8700cfd 
Co-authored-by: Michal Fiedorowicz <[email protected]>
@arthanson
OBS-1046 update message
7c07bfe
@ltucker
feat: client credentials management (#106)
5721150
@github-actions
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
3227 2879 89% 0% 🟢

New Files

File Coverage Status
netbox_diode_plugin/client.py 39% 🟢
netbox_diode_plugin/diode/clients.py 70% 🟢
netbox_diode_plugin/tables.py 77% 🟢
netbox_diode_plugin/tests/test_diode_clients.py 100% 🟢
TOTAL 72% 🟢

Modified Files

File Coverage Status
netbox_diode_plugin/init.py 100% 🟢
netbox_diode_plugin/forms.py 100% 🟢
netbox_diode_plugin/models.py 100% 🟢
netbox_diode_plugin/navigation.py 100% 🟢
netbox_diode_plugin/plugin_config.py 68% 🟢
netbox_diode_plugin/urls.py 100% 🟢
netbox_diode_plugin/views.py 55% 🟢
TOTAL 89% 🟢

updated for commit: 5721150 by action🐍

@ltucker ltucker merged commit 84824cf into release May 16, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mfiedorowicz mfiedorowicz mfiedorowicz approved these changes

@leoparente leoparente Awaiting requested review from leoparente leoparente is a code owner

Assignees

No one assigned

Labels

internal markdown python

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ltucker @mfiedorowicz @arthanson