[gssapi/gssproxy] db71ef: Revert break_loop changes
by Simo Sorce
Branch: refs/heads/main
Home: https://github.com/gssapi/gssproxy
Commit: db71ef93d714163beefcb1a28046188a6642cb6c
https://github.com/gssapi/gssproxy/commit/db71ef93d714163beefcb1a28046188...
Author: Simo Sorce <simo(a)redhat.com>
Date: 2022-05-26 (Thu, 26 May 2022)
Changed paths:
M src/gp_init.c
M src/gp_proxy.h
M src/gssproxy.c
Log Message:
-----------
Revert break_loop changes
The event loop cycles too often to use verto_run_once() as a way
to deal with marking activity. So revert to using verto_run() and
set the termination handler once for now.
Signed-off-by: Simo Sorce <simo(a)redhat.com>
Commit: 7d771bd6dd8bef4b7b5f4b4f29c9a8834ec49b25
https://github.com/gssapi/gssproxy/commit/7d771bd6dd8bef4b7b5f4b4f29c9a88...
Author: Simo Sorce <simo(a)redhat.com>
Date: 2022-05-26 (Thu, 26 May 2022)
Changed paths:
M src/gp_init.c
M src/gp_proxy.h
M src/gssproxy.c
Log Message:
-----------
Move initialization functions into gp_init
Refactor init functions and move them in the appropriate file.
Signed-off-by: Simo Sorce <simo(a)redhat.com>
Commit: f48f8c6b92bdbbc2ea7ce8b8d9050e2a2195579e
https://github.com/gssapi/gssproxy/commit/f48f8c6b92bdbbc2ea7ce8b8d9050e2...
Author: Simo Sorce <simo(a)redhat.com>
Date: 2022-05-26 (Thu, 26 May 2022)
Changed paths:
M Makefile.am
M src/gp_init.c
A src/gp_mgmt.c
M src/gp_proxy.h
M src/gssproxy.c
Log Message:
-----------
Move runtime management function to gp_mgmt.c
These are functions that are used to manage the proxy own main process
after initialization. Store in their own file so the scope is clear.
Signed-off-by: Simo Sorce <simo(a)redhat.com>
Commit: 0400a19aefe6d19d4d187bc6f3038591e5bde77e
https://github.com/gssapi/gssproxy/commit/0400a19aefe6d19d4d187bc6f303859...
Author: Simo Sorce <simo(a)redhat.com>
Date: 2022-05-26 (Thu, 26 May 2022)
Changed paths:
M src/gp_debug.h
M src/gp_mgmt.c
M src/gp_proxy.h
M src/gp_socket.c
Log Message:
-----------
Fix self termination and add some accounting
Self termination need to happen only when gssproxy is idle for a certina
amount of time.
However due to non material activity the gssproxy performs through the
verto event handler we need to actively measure ideling rather than
depend on things like verto_idle.
Ad an accounting function to our socket handler.
This now allows us to collect some interesting statistics as well as
reset the idle handler when there is actual activity ongoing.
We reset the handler only upon receiving data from a client, as that
indicates a requested activity. Writes happen only as reposnses and are
considered part of a single activity event together with the read that
caused the idel handler reset.
Signed-off-by: Simo Sorce <simo(a)redhat.com>
Commit: a3d215e24237778681965eacaa42a596cd3842df
https://github.com/gssapi/gssproxy/commit/a3d215e24237778681965eacaa42a59...
Author: Simo Sorce <simo(a)redhat.com>
Date: 2022-05-26 (Thu, 26 May 2022)
Changed paths:
M tests/userproxytest.c
Log Message:
-----------
Extend userproxy test to check outputs
Currently just checks that initialization and self termination
actually happened.
Signed-off-by: Simo Sorce <simo(a)redhat.com>
Compare: https://github.com/gssapi/gssproxy/compare/c8b9107288f9...a3d215e24237
1 year, 4 months
[gssapi/gssproxy] c8b910: Fix issues with socket activation.
by Simo Sorce
Branch: refs/heads/main
Home: https://github.com/gssapi/gssproxy
Commit: c8b9107288f9de4650612074efcca592eb6a2f0c
https://github.com/gssapi/gssproxy/commit/c8b9107288f9de4650612074efcca59...
Author: Simo Sorce <simo(a)redhat.com>
Date: 2022-05-24 (Tue, 24 May 2022)
Changed paths:
M src/gp_config.c
M src/gp_proxy.h
M src/gp_socket.c
M src/gssproxy.c
M systemd/gssuserproxy.service.in
Log Message:
-----------
Fix issues with socket activation.
Adjust systemd's service file for usermode.
Socket activation requires that we used the interactive mode (-i)
so that gssproxy does not fork on startup.
Forking causes sd_listen_fds() to fail as that function checks that
the pid of the process calling it is the same process systemd forked
via the internal envvar LISTEN_PID.
Additionally make sure to set the socket name for the artificial
config service in usermode as well as the socket set up during socket
activation to all use the same string (the usermode socket) so that
the checks in gp_creds_match_conn() work properly in the socket
activation case too.
Signed-off-by: Simo Sorce <simo(a)redhat.com>
1 year, 4 months