# Node-red-node-aws: html form state bucket is required which prevent providing from msg

**URL:** <https://discourse.nodered.org/t/node-red-node-aws-html-form-state-bucket-is-required-which-prevent-providing-from-msg/71165>\
**Category:** General\
**Created:** [24 November 2022 09:45 UTC](https://discourse.nodered.org/t/node-red-node-aws-html-form-state-bucket-is-required-which-prevent-providing-from-msg/71165 "2022-11-24T09:45:25Z")\
**Posts on this page:** 6\
**Page:** 1

<div class="post-metadata">

**Author:** ![Barbudor](https://avatars.discourse-cdn.com/v4/letter/b/4af34b/32.png) [@Barbudor](https://discourse.nodered.org/u/Barbudor)\
**Post date:** [24 November 2022 09:45 UTC](https://discourse.nodered.org/t/node-red-node-aws-html-form-state-bucket-is-required-which-prevent-providing-from-msg/71165/1 "2022-11-24T09:45:25Z")

</div>

Hi

The AWS nodes "Amazon S3" and Amazon S3 out" can support a bucket name provided as the msg.bucket property. It works fine but as the form specify that the bucket field in the form is required, a warning is generated on each deploy.

I propose a PR to change those 2 descriptors from:

```auto
            bucket: {required:true},

```

to

```auto
            bucket: {value:""},

```

in

> <https://github.com/node-red/node-red-web-nodes/blob/52cdc1bb29ed8757295300371912ef9ff5a84a44/aws/aws.html#L129>

and

> <https://github.com/node-red/node-red-web-nodes/blob/52cdc1bb29ed8757295300371912ef9ff5a84a44/aws/aws.html#L66>

I'm ready to send the PR if you are ok

Just for your reference, the JS code is perfectly handling that:

> <https://github.com/node-red/node-red-web-nodes/blob/52cdc1bb29ed8757295300371912ef9ff5a84a44/aws/aws.js#L162>

and

> <https://github.com/node-red/node-red-web-nodes/blob/52cdc1bb29ed8757295300371912ef9ff5a84a44/aws/aws.js#L220>

Thanks

---

<div class="post-metadata">

**Author:** ![dceejay](https://sea2.discourse-cdn.com/flex026/user_avatar/discourse.nodered.org/dceejay/32/38_2.png) [@dceejay](https://discourse.nodered.org/u/dceejay)\
**Post date:** [24 November 2022 10:12 UTC](https://discourse.nodered.org/t/node-red-node-aws-html-form-state-bucket-is-required-which-prevent-providing-from-msg/71165/2 "2022-11-24T10:12:51Z")

</div>

yes - please raise a PR.

---

<div class="post-metadata">

**Author:** ![Barbudor](https://avatars.discourse-cdn.com/v4/letter/b/4af34b/32.png) [@Barbudor](https://discourse.nodered.org/u/Barbudor)\
**Post date:** [24 November 2022 11:11 UTC](https://discourse.nodered.org/t/node-red-node-aws-html-form-state-bucket-is-required-which-prevent-providing-from-msg/71165/3 "2022-11-24T11:11:58Z")

</div>

Thanks @dceejay  
Done [AWS - bucket not required in form for s3query and s3out by jeanmichel-nwsb · Pull Request #303 · node-red/node-red-web-nodes · GitHub](https://github.com/node-red/node-red-web-nodes/pull/303)

Best regards

---

<div class="post-metadata">

**Author:** ![dceejay](https://sea2.discourse-cdn.com/flex026/user_avatar/discourse.nodered.org/dceejay/32/38_2.png) [@dceejay](https://discourse.nodered.org/u/dceejay)\
**Post date:** [24 November 2022 11:29 UTC](https://discourse.nodered.org/t/node-red-node-aws-html-form-state-bucket-is-required-which-prevent-providing-from-msg/71165/4 "2022-11-24T11:29:33Z")

</div>

Published as v0.2.3. Thanks

---

<div class="post-metadata">

**Author:** ![Barbudor](https://avatars.discourse-cdn.com/v4/letter/b/4af34b/32.png) [@Barbudor](https://discourse.nodered.org/u/Barbudor)\
**Post date:** [24 November 2022 12:28 UTC](https://discourse.nodered.org/t/node-red-node-aws-html-form-state-bucket-is-required-which-prevent-providing-from-msg/71165/5 "2022-11-24T12:28:39Z")

</div>

Thanks a lot

---

<div class="post-metadata">

**Author:** ![system](https://us1.discourse-cdn.com/flex026/uploads/nodered/original/1X/d073cd938eafa2e558d7c2cd59003b3ef4963033.png) [@system](https://discourse.nodered.org/u/system)\
**Post date:** [23 January 2023 12:29 UTC](https://discourse.nodered.org/t/node-red-node-aws-html-form-state-bucket-is-required-which-prevent-providing-from-msg/71165/6 "2023-01-23T12:29:20Z")

</div>

This topic was automatically closed 60 days after the last reply. New replies are no longer allowed.