improvement: do not require basic password in UI (#5812)
* Fix basic-auth.jsx: do not require password in UI password is not PropTypes.string.isRequired, but the markup wrongly makes it mandatory * Do not require password input in http-auth.jsx This is used by the Authorize modal. Not sure when the similar code from basic-auth.jsx is used. Co-authored-by: kyle shockey <kyle.shockey1@gmail.com>
This commit is contained in:
@@ -73,8 +73,7 @@ export default class BasicAuth extends React.Component {
|
|||||||
<label>Password:</label>
|
<label>Password:</label>
|
||||||
{
|
{
|
||||||
username ? <code> ****** </code>
|
username ? <code> ****** </code>
|
||||||
: <Col><Input required="required"
|
: <Col><Input autoComplete="new-password"
|
||||||
autoComplete="new-password"
|
|
||||||
name="password"
|
name="password"
|
||||||
type="password"
|
type="password"
|
||||||
onChange={ this.onChange }/></Col>
|
onChange={ this.onChange }/></Col>
|
||||||
|
|||||||
@@ -81,8 +81,7 @@ export default class HttpAuth extends React.Component {
|
|||||||
<label>Password:</label>
|
<label>Password:</label>
|
||||||
{
|
{
|
||||||
username ? <code> ****** </code>
|
username ? <code> ****** </code>
|
||||||
: <Col><Input required="required"
|
: <Col><Input autoComplete="new-password"
|
||||||
autoComplete="new-password"
|
|
||||||
name="password"
|
name="password"
|
||||||
type="password"
|
type="password"
|
||||||
onChange={ this.onChange }/></Col>
|
onChange={ this.onChange }/></Col>
|
||||||
|
|||||||
Reference in New Issue
Block a user