Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Webcontainer property decodeUrlPlusSign issue #12074

Closed
epj opened this issue May 6, 2020 · 3 comments · Fixed by #12085
Closed

Webcontainer property decodeUrlPlusSign issue #12074

epj opened this issue May 6, 2020 · 3 comments · Fixed by #12085
Assignees
Labels
in:Web Components POC Approved release bug This bug is present in a released version of Open Liberty release:20008 team:Sirius

Comments

@epj
Copy link
Contributor

epj commented May 6, 2020

Currently for decodeUrlPlusSign, false setting is ignored.

There also needs to be a doc update, page rwlp_metatype_4ic currently says default for decodeUrlPlusSign is "false", but its actually always been "true".

@epj epj added in:Web Components release bug This bug is present in a released version of Open Liberty team:Sirius labels May 6, 2020
@epj epj self-assigned this May 6, 2020
@chirp1
Copy link
Contributor

chirp1 commented May 12, 2020

@epj Hi Evan, see my slack to you. I believe the file that you want to update is https://www.ibm.com/support/knowledgecenter/SSAW57_liberty/com.ibm.websphere.liberty.autogen.nd.doc/ae/rwlp_config_webContainer.html , which is a Liberty autogen topic that you would update. It contains the decodeUrlPlusSign element that you mention.

@pnicolucci
Copy link
Member

@epj can you update the issue here since a lot has changed since you first opened?

@mbroz2
Copy link
Member

mbroz2 commented Jun 11, 2020

The POC Vote for "Do we agree to fix the decodeUrlPlusSign config property so it takes affect, and change its default to "true" to maintain the existing behavior?" was APPROVED.

Web Tier Team automation moved this from General Issues to Completed Tasks Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in:Web Components POC Approved release bug This bug is present in a released version of Open Liberty release:20008 team:Sirius
Projects
Web Tier Team
  
Completed Tasks
Development

Successfully merging a pull request may close this issue.

5 participants