Skip to content

Add digest authorization and TCP protocol#32

Open
alexgor1 wants to merge 2 commits into
jart:masterfrom
alexgor1:master
Open

Add digest authorization and TCP protocol#32
alexgor1 wants to merge 2 commits into
jart:masterfrom
alexgor1:master

Conversation

@alexgor1
Copy link
Copy Markdown

@alexgor1 alexgor1 commented Jan 4, 2024

  • add digest authorization,
  • add support of tcp protocol,
  • add support of empty Contact header in message,

- add support of tcp protocol,
- add support of empty Contact header in message,
@negbie
Copy link
Copy Markdown
Collaborator

negbie commented Jan 4, 2024

Hi Alex, can you please update the go.sum file?

@alexgor1
Copy link
Copy Markdown
Author

alexgor1 commented Jan 4, 2024

Hi Alex, can you please update the go.sum file?

updated

@jart
Copy link
Copy Markdown
Owner

jart commented Jan 5, 2024

Looks good to me overall. Is there any chance I could persuade you to vendor the HTTP digest code? What it's doing is simple enough to not merit a dependency. I'd rather just vendor the needed code in this project, and add a copyright header to the tops of the file(s) to credit the original author(s).

@alexgor1
Copy link
Copy Markdown
Author

alexgor1 commented Jan 5, 2024

Looks good to me overall. Is there any chance I could persuade you to vendor the HTTP digest code? What it's doing is simple enough to not merit a dependency. I'd rather just vendor the needed code in this project, and add a copyright header to the tops of the file(s) to credit the original author(s).

It's your project and it's up to you how to include changes to it, so you can vendor digest code of course. As for me i don't like to duplicate existed code which has already been tested. Moreover it's much harder to maintain it and keep up to date. For example last update to digest project on github dated 3 months ago. Anyway i need authorization functionality for SIP stack and if you want to add it into your code and maintain it then it can work for me I guess. I've just offered simplest and clear solution which i use in my fork.

Comment thread dialog/receiver.go
Copy link
Copy Markdown

@razor-1 razor-1 left a comment

Choose a reason for hiding this comment

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

TCP support should be implemented in a way that complies with the RFC

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.

4 participants