Project

General

Profile

Bug #33440

"OAuth map users" setting has a bad explanation

Added by Dominik Matoulek 11 months ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Category:
Settings
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

In settings, there is a "OAuth map users" where explanation says "Foreman will map users by username in request-header. If this is set to false, OAuth requests will have admin rights."

The problem is, that the settings has options "Yes" and "No". This can be confusing for users.

I found that during fixing of #33439.


Related issues

Related to Katello - Bug #33439: Foreman setting has wrong description for a couple of parameters under Content Tab on the Foreman WebUIClosed

Associated revisions

Revision f25a7620 (diff)
Added by Dominik Matoulek 11 months ago

Fixes #33440 - improve description for "Oauth map users" setting

This setting has "Yes" and "No" options in UI so description should not use true/false.

History

#1 Updated by The Foreman Bot 11 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8773 added

#2 Updated by Dominik Matoulek 11 months ago

  • Related to Bug #33439: Foreman setting has wrong description for a couple of parameters under Content Tab on the Foreman WebUI added

#3 Updated by The Foreman Bot 11 months ago

  • Fixed in Releases 3.2.0 added

#4 Updated by Anonymous 11 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by Ewoud Kohl van Wijngaarden 11 months ago

  • Fixed in Releases 3.1.0 added
  • Fixed in Releases deleted (3.2.0)
  • Pull request deleted (https://github.com/theforeman/foreman/pull/8773)

#6 Updated by Ewoud Kohl van Wijngaarden 11 months ago

  • Target version set to 3.1.0
  • Subject changed from "OAuth map users" setting has bad explanation to "OAuth map users" setting has a bad explanation
  • Fixed in Releases deleted (3.1.0)
  • Pull request https://github.com/theforeman/foreman/pull/8773 added

#7 Updated by Ewoud Kohl van Wijngaarden 11 months ago

  • Fixed in Releases 3.1.0 added

Also available in: Atom PDF