]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
note recent TODO's
authorAlan T. DeKok <aland@freeradius.org>
Thu, 24 Aug 2017 20:16:02 +0000 (16:16 -0400)
committerAlan T. DeKok <aland@freeradius.org>
Thu, 24 Aug 2017 20:25:25 +0000 (16:25 -0400)
src/modules/rlm_radius/TODO.md

index 90cc68770976233a4fa1730e04bc99d293828b12..31a3021b09d9a0355b63b1a2766a25f364b3d561 100644 (file)
@@ -40,36 +40,6 @@ We should limit the number of outgoing connections, tho.
 
 ## synchronous proxying
 
-ala v3.  All retransmissions started by the client.
-
-This is probably trivial as a side-effect of doing de-dup /
-conflicting packet in the server core.
-
-This requires a "signal" handler to be added when the module calls
-unlang_yield.
-
-The call to the signal handler is already in proto_radius_auth and
-friends.
-
-We should add a signal handler to the module, to handle the DUP / DONE
-signals.  This would allow graceful cleanups.  Those are mostly
-already handled via the talloc_free() hierarchy and destructors. But
-it may be nice to distinguish the situations.  And, it lets us test
-the signal handler independent of anything else.
-
-Doing synchronous proxying also mean having the network side return
-DUP PACKET (somehow).  And, send that dup packet signal to the worker.
-Which somehow associates it with a request (probably via a simple
-network thread + packet identifier).  This means that the worker has
-to have yet another tree tracking packets... but it will allow for
-signaling if necessary.
-
-We probaby want the network + worker to be able to send IDs of 0/0,
-which means "no tracking", as that will likely be the common case.
-
-We also need the same thing for conflicting packets... we need a way
-to tell the end modules to stop retransmitting the packet, as no one
-cares about it any more.
 
 ## miscellaneous
 
@@ -134,4 +104,14 @@ for Access-Request, and just username for Accounting-Request.
   child REQUEST async stuff with listen, protocol handler, etc.
 
 * fork also needs to do this sanity check on compile, so that it knows
-  it can dereference things which exist...
\ No newline at end of file
+  it can dereference things which exist...
+
+## connection state
+
+* CONN_UNUSED isn't used...
+
+* CONN_STATUS_CHECKS isn't used...that needs to be fixed
+
+* add configurable timers for response_window (see @todo in rlm_radius_udp)
+
+*
\ No newline at end of file