First Last Prev Next    This bug is not in your last search results.
Bug 49831 - ucs-school-radius-802.1x: Update to 4.4 does not remove unused UCR variables
ucs-school-radius-802.1x: Update to 4.4 does not remove unused UCR variables
Status: NEW
Product: UCS@school
Classification: Unclassified
Component: Radius
UCS@school 4.4
Other Windows NT
: P5 minor (vote)
: ---
Assigned To: UCS@school maintainers
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2019-07-10 13:02 CEST by Mathieu Simon
Modified: 2023-06-23 13:43 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mathieu Simon 2019-07-10 13:02:07 CEST 
Hi

With UCS 4.4 the big part of ucs-school-radius-802.1x has been merged with univention-radius in order to reduce code duplication[1] which is a good thing.

For what I can tell, the following variables are now obsolete:
security/packetfilter/package/ucs-school-radius-802.1x/udp/1812/all/en
security/packetfilter/package/ucs-school-radius-802.1x/udp/1812/all
security/packetfilter/package/ucs-school-radius-802.1x/udp/1813/all/en
security/packetfilter/package/ucs-school-radius-802.1x/udp/1813/all
security/packetfilter/package/ucs-school-radius-802.1x/udp/1814/all/en
security/packetfilter/package/ucs-school-radius-802.1x/udp/1814/all

They are the duplicates of:
security/packetfilter/package/univention-radius/udp/1812/all/en
security/packetfilter/package/univention-radius/udp/1812/all
security/packetfilter/package/univention-radius/udp/1813/all/en
security/packetfilter/package/univention-radius/udp/1813/all
security/packetfilter/package/univention-radius/udp/1814/all/en
security/packetfilter/package/univention-radius/udp/1814/all

The postinst and postrm of univention-radius properly handles the later addition and removal of these UCR variables (as did ucs-school-radius-802.1x previously as per git history). However upgraded systems will have ucs-school-radius-802.1x variables until someone manually cleans them with 'ucr unset'.

Could you release an update to ucs-school-radius-802.1x that cleanses these unused variables if they are found?

Regards
Mathieu

[1] https://github.com/univention/ucs-school/commit/675cc80b90c60f3243787ded9c6d1b51709d42b4#diff-b6f6261ecbaa2db21e62f92f9259cd71
Comment 1 Mathieu Simon 2019-07-18 15:27:42 CEST 
Addittionnaly I've discovered that dpkg still references the template for eap.conf from FreeRADIUS 2.2.x in ucs-school-radius-802.1x version 7.0.1-6A~4.4.0.201902270211

dpkg -S /etc/univention/templates/files/etc/freeradius/eap.conf
ucs-school-radius-802.1x: /etc/univention/templates/files/etc/freeradius/eap.conf

I've given a look at the code on github and there is no file or reference that I could spot, maybe a build error?

Looking forward to these cleanups.
Comment 2 Mathieu Simon 2019-07-18 15:31:26 CEST 
After cleaning up other things, it appears that ldap.attrmap and radiusd.conf are also affected.
First Last Prev Next    This bug is not in your last search results.