Ticket #2444 enhancement closed wontfix

Opened 6 years ago

Last modified 4 years ago

Merge Divmod.Sine sip.py into twisted.protocols.sip

Reported by: therve Owned by: washort
Priority: normal Milestone:
Component: core Keywords:
Cc: washort, itamarst, thijs Branch:
Author: Launchpad Bug:

Description

Maintenance has been done in Divmod repository for years. We should merge the code, but that requires to remove references to other Divmod projects (at least axiom and epsilon). Also tests should probably be updated.

Change History

1

Changed 6 years ago by itamarst

  • cc itamarst added

Awesome, glad you're doing this. Random notes on current pass, though I'm sure you're already aware of some:

1. Coding standard violations (use of _). 2. Replace exarkun@… with @example.com addresses. 3. Are we aiming for backwards compat with old code, or just assuming no one used it? 4. The comments in respondToAuthChallenge about bugs and the hardcoded asterisk stuff are disturbing :) Can you expand on this, washort?

2

Changed 5 years ago by thijs

  • owner changed from therve to washort
  • cc thijs added

Assigning back to washort, and wondering where this code can be found?

3

Changed 5 years ago by exarkun

#3533 was a duplicate of this.

4

Changed 4 years ago by washort

  • status changed from new to closed
  • resolution set to wontfix

This isn't going to happen, exactly. The code in Sine is poorly tested and doesn't conform to the spec in certain ways.The transport and transaction layers have been reimplemented in #3584. The proxy and registrar will be done under #3662.

Note: See TracTickets for help on using tickets.