Skip to content

Commit 78c24f6

Browse files
author
Emmanouil Konstantinidis
committed
Hide headers if 'AllowAny'
1 parent a721630 commit 78c24f6

File tree

5 files changed

+29
-13
lines changed

5 files changed

+29
-13
lines changed

rest_framework_docs/static/rest_framework_docs/js/components/request.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,10 @@ var Request = React.createClass({
8080
selectedMethod={this.state.selectedMethod}
8181
setMethod={this.setSelectedMethod} />
8282

83-
<Header title='Headers' />
84-
<Headers headers={this.state.headers} handleHeaderChange={this.handleHeaderChange} />
83+
<Headers
84+
headers={this.state.headers}
85+
permissions={this.state.endpoint.permissions}
86+
handleHeaderChange={this.handleHeaderChange} />
8587

8688
{RequestUtils.shouldAddData(this.state.method) ? null : (
8789
<div>

rest_framework_docs/static/rest_framework_docs/js/components/request/headers.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
var React = require('react');
22

3+
var Header = require('../helpers/header');
34
var Input = require('../helpers/input');
5+
var RequestUtils = require('../../utils/request');
46

57
var Headers = React.createClass({
68

@@ -21,8 +23,13 @@ var Headers = React.createClass({
2123
},
2224

2325
render: function () {
26+
if (!RequestUtils.shouldAddHeader(this.props.permissions)) {
27+
return null;
28+
}
29+
2430
return (
2531
<div>
32+
<Header title='Headers' />
2633
<Input
2734
name='authorization'
2835
value={this.state.authorization}

rest_framework_docs/static/rest_framework_docs/js/dist.js

Lines changed: 15 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -32549,8 +32549,10 @@ var Request = React.createClass({
3254932549
methods: this.state.endpoint.methods,
3255032550
selectedMethod: this.state.selectedMethod,
3255132551
setMethod: this.setSelectedMethod }),
32552-
React.createElement(Header, { title: 'Headers' }),
32553-
React.createElement(Headers, { headers: this.state.headers, handleHeaderChange: this.handleHeaderChange }),
32552+
React.createElement(Headers, {
32553+
headers: this.state.headers,
32554+
permissions: this.state.endpoint.permissions,
32555+
handleHeaderChange: this.handleHeaderChange }),
3255432556
RequestUtils.shouldAddData(this.state.method) ? null : React.createElement(
3255532557
'div',
3255632558
null,
@@ -32646,7 +32648,9 @@ module.exports = FieldsData;
3264632648

3264732649
var React = require('react');
3264832650

32651+
var Header = require('../helpers/header');
3264932652
var Input = require('../helpers/input');
32653+
var RequestUtils = require('../../utils/request');
3265032654

3265132655
var Headers = React.createClass({
3265232656
displayName: 'Headers',
@@ -32668,9 +32672,14 @@ var Headers = React.createClass({
3266832672
},
3266932673

3267032674
render: function render() {
32675+
if (!RequestUtils.shouldAddHeader(this.props.permissions)) {
32676+
return null;
32677+
}
32678+
3267132679
return React.createElement(
3267232680
'div',
3267332681
null,
32682+
React.createElement(Header, { title: 'Headers' }),
3267432683
React.createElement(Input, {
3267532684
name: 'authorization',
3267632685
value: this.state.authorization,
@@ -32682,7 +32691,7 @@ var Headers = React.createClass({
3268232691

3268332692
module.exports = Headers;
3268432693

32685-
},{"../helpers/input":165,"react":159}],171:[function(require,module,exports){
32694+
},{"../../utils/request":175,"../helpers/header":164,"../helpers/input":165,"react":159}],171:[function(require,module,exports){
3268632695
'use strict';
3268732696

3268832697
var React = require('react');
@@ -32853,7 +32862,6 @@ var LiveAPIEndpoints = require('./components/liveapi');
3285332862

3285432863
var utils = {
3285532864
transformMethods: function transformMethods(methods) {
32856-
console.log(methods);
3285732865
return methods.replace(/\W+/g, ' ').replace(/^[ ]+|[ ]+$/g, '').split(' ');
3285832866
}
3285932867
};
@@ -32913,10 +32921,10 @@ module.exports = {
3291332921
},
3291432922

3291532923
shouldAddHeader: function shouldAddHeader(permissions) {
32916-
if (permissions === 'AllowAny' || permissions === 'None') {
32917-
return true;
32924+
if (permissions === 'AllowAny') {
32925+
return false;
3291832926
}
32919-
return false;
32927+
return true;
3292032928
}
3292132929

3292232930
};

rest_framework_docs/static/rest_framework_docs/js/index.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ var LiveAPIEndpoints = require('./components/liveapi');
99

1010
var utils = {
1111
transformMethods: function (methods) {
12-
console.log(methods);
1312
return methods
1413
.replace(/\W+/g, ' ')
1514
.replace(/^[ ]+|[ ]+$/g,'')

rest_framework_docs/static/rest_framework_docs/js/utils/request.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ module.exports = {
88
},
99

1010
shouldAddHeader: function (permissions) {
11-
if (permissions === 'AllowAny' || permissions === 'None') {
12-
return true;
11+
if (permissions === 'AllowAny') {
12+
return false;
1313
}
14-
return false;
14+
return true;
1515
}
1616

1717
};

0 commit comments

Comments
 (0)