Skip to content
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

Changes to nclient4 and nclient6 from our testing phase #548

Closed
wants to merge 6 commits into from

Conversation

jrsmroz
Copy link

@jrsmroz jrsmroz commented Sep 2, 2024

This PR starts Vitirify fork of insomniacslk/dhcp.

Changes include:

  • Ability to send relay message from nclient6
  • Ability to set read buffor size in nclient4

Core functionality of dhcp message handling was not touches.

@@ -1,3 +1,5 @@
replace github.com/insomniacslk/dhcp => gitlab.com/Vitrifi/insomniacslk-dhcp v0.0.1-cne
Copy link
Collaborator

Choose a reason for hiding this comment

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

This doesn't look right

Copy link
Author

Choose a reason for hiding this comment

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

Hello. Thank you for comment. It's what we used internally to get it working quick and dirty. It'm in a process of cleaning that up.

As is it may not meet open source project standards yet.

Copy link
Author

Choose a reason for hiding this comment

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

By the way, that Pull Request was supposed to be inside our fork, but it seems I've done it wrongly.

@pmazzini
Copy link
Collaborator

pmazzini commented Sep 2, 2024

Changes include:
Ability to send relay message from nclient6
Ability to set read buffor size in nclient4

Maybe better to split in different PRs.

@jrsmroz jrsmroz force-pushed the cne-rebase-to-master branch from cdb1fe0 to abaae81 Compare September 2, 2024 13:12
@jrsmroz
Copy link
Author

jrsmroz commented Sep 2, 2024

Closing this pull request. It was not supposed to be opened into insomniacslk:master yet. We will attempt to commit our changes back after the cleanup. Once again, I'm sorry for the confusion.

@jrsmroz jrsmroz closed this Sep 2, 2024
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.

2 participants