Ticket #612 (closed Bug report: fixed)

Opened 5 years ago

Last modified 5 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 5 years ago by gbastien

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

Corrected in [1438]

Note: See TracTickets for help on using tickets.