-
Notifications
You must be signed in to change notification settings - Fork 18
feat: send data to new enterprise portal proxy #1509
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Adds temporary “migration proxy” POSTs so enterprise systems also send heartbeat and inventory in the new phonehome format to my.nethesis.it/proxy/*, supporting the enterprise portal migration workflow.
Changes:
- Send an additional enterprise-only heartbeat POST to
https://my.nethesis.it/proxy/heartbeat. - Send an additional enterprise-only inventory POST (phonehome JSON) to
https://my.nethesis.it/proxy/inventory. - Extend the phonehome inventory payload with extra
inventory.info_*facts.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| packages/ns-plug/files/send-heartbeat | Adds enterprise-only proxy heartbeat POST after existing heartbeat send. |
| packages/ns-phonehome/files/send-phonehome | Adds enterprise-only proxy inventory POST using basic auth after existing phonehome send. |
| packages/ns-phonehome/files/phonehome | Adds inventory.info_* data into the generated facts payload. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@gsanchietti I've opened a new pull request, #1516, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@gsanchietti I've opened a new pull request, #1517, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@gsanchietti I've opened a new pull request, #1518, to work on those changes. Once the pull request is ready, I'll request review from you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
9240363 to
db77a86
Compare
The new my.nethesis.it portal should start receiving data for enterprise subscription: this a temporary change that will help the migration to th new platform Add also new info to phonehome to mimic inventory of NethServer 8
db77a86 to
ce96983
Compare
The new my.nethesis.it portal should start receiving data for enterprise subscription: this a temporary change that will help the migration to the new platform
Implements: #1512
Requires: NethServer/python3-nethsec#113