-
Notifications
You must be signed in to change notification settings - Fork 79
Setting butler request size limit for 50mb by default #1228
Setting butler request size limit for 50mb by default #1228
Conversation
Signed-off-by: Lucas Borges Fernandes <lucasbfernandes94@gmail.com>
Codecov Report
@@ Coverage Diff @@
## charlescd-release-1.0.0 #1228 +/- ##
==========================================================
Coverage 76.89% 76.89%
Complexity 2177 2177
==========================================================
Files 1307 1306 -1
Lines 18736 18718 -18
Branches 1711 1710 -1
==========================================================
- Hits 14407 14394 -13
+ Misses 3826 3821 -5
Partials 503 503
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Signed-off-by: Lucas Borges Fernandes <lucasbfernandes94@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 comments on helm chart
@@ -110,6 +110,8 @@ env: | |||
value: "http://charlescd-butler.{{ .ChartContext.Release.Namespace }}.svc.cluster.local:3000" | |||
- name: BUTLER_NAMESPACE | |||
value: {{ .ChartContext.Release.Namespace }} | |||
- name: REQUEST_SIZE_LIMIT | |||
value: {{ .RangeContext.requestSizeLimit }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The right way is passing the value in quotes:
value: {{ .RangeContext.requestSizeLimit | default 50mb| quote}}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed here 51713b4
install/helm-chart/values.yaml
Outdated
@@ -46,6 +46,7 @@ CharlesApplications: | |||
password: 3f2Yq8R4HhDCnefR | |||
name: charlescd_butler | |||
ssl: false | |||
requestSizeLimit: '50mb' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right way is:
requestSizeLimit: 50mb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed here 51713b4
Signed-off-by: Lucas Borges Fernandes <lucasbfernandes94@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fix: Request entity too large caused by large payloads sent from metacontroller.