Ticket #612 (closed Bug report: fixed)

Opened 5 years ago

Last modified 4 years ago

Radius fails in various ways if there are spaces in the authenticator constructor

Reported by: steven.kurylo@… Owned by:
Priority: normal Milestone: Not yet assigned to a Milestone
Component: Auth server, Authentication, permissions and access control Version:
Keywords: radius Cc:

Description

Depending on where you put a space in the constructor, radius fails in various ways with no indication that the space is the problem.

So let's strip out the spaces before saving the constructor, as a space isn't valid in any of the fields. The also aligns the documentation (which has spaces) and what will work.

Attachments

radius_spaces.diff Download (0.6 KB) - added by steven.kurylo@… 5 years ago.
radius_spaces.2.diff Download (504 bytes) - added by steven.kurylo@… 5 years ago.
trim done in during parsing instead of saving

Change History

Changed 5 years ago by steven.kurylo@…

Changed 5 years ago by steven.kurylo@…

trim done in during parsing instead of saving

Changed 5 years ago by steven.kurylo@…

So maybe the second version of the patch is better - its doing trim where the authenticator construct is being parsed.

Changed 4 years ago by gbastien

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

Corrected in [1438]

Add/Change #612 (Radius fails in various ways if there are spaces in the authenticator constructor)

Author


E-mail address and user name can be saved in the Preferences.


Action
as closed
The resolution will be deleted. Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.