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

Merge master (v1.33.2) into develop #1158

Merged
merged 13 commits into from
Dec 8, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 8, 2023

Merge master (v1.33.2) into develop

cea2aj and others added 13 commits July 26, 2023 15:44
## v1.32.0

### Changes

- Support rendering of HTML for richTextV2 and markdown direct answers (#1138)
## v1.33.0

### Changes

- Update FAQ card to support answerV2 instead of answer (#1142)
- Update corresponding translation tests for above changes (#1147)
Iframe needs height in order for map and toggle button to be initialized and to fill js-answersMap
Fix iframe Answers Map/List toggle button on Mobile Error
Currently, referrerPageUrl is added as a query param value without being escaped. That can cause problems on sites with tighter security restrictions. We faced this problem on several sites such as this one: https://yexttest.atlassian.net/browse/PC-208916

It can also cause functional problems when the referrer URL has query parameters. For example, a URL of http://example.com?a=1&b=2 will lead to https://answers.example.com?query=TEST&referrerPageUrl=http://example.com?a=1&b=2

In that example, &b=2 will be treated as a query parameter of the  results page URL instead of being part of the referrerPageUrl value.
fix(iframe-common.js): properly encode referrerPageUrl param
* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map

* Implement playwright acceptance tests for vertical full page map
Upgrade `@babel/traverse` from v7.17.9 (in top-level package.json) and v7.10.4 (in static/package.json) to 7.23.5 to address a critical vulnerability in the package. Also, remove Browserstack dependencies and references that are no longer needed.

J=VULN-38474, VULN-38475
TEST=auto
### Fixes
- Added encoding for the `referrerPageUrl` param so it doesn't affect the main URL's query params and works better with some tighter security restrictions (#1152)
- Upgraded `@babel/traverse` version to address a vulnerability in the package (#1156)
@nmanu1 nmanu1 merged commit 7aff0d8 into develop Dec 8, 2023
16 checks passed
@nmanu1 nmanu1 deleted the dev/merge-v1.33.2-c66bc3f-into-develop branch December 8, 2023 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants