Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

FLOW-1037 lineage node template rendering logic updated to fix html i… #214

Merged
merged 3 commits into from
Dec 19, 2023
Merged

FLOW-1037 lineage node template rendering logic updated to fix html i… #214

merged 3 commits into from
Dec 19, 2023

Conversation

vikas-cldcvr
Copy link
Contributor

…njection

Checklist for raising a PR

  • Gone through UX documnetation for adding new features.
  • All necessary unit tests covered.
  • Required comments added for generating component manifest file? you can find details here
  • Did you check the contributing doc?
  • Did you check the existing issues for similar queries?

Describe your PR

@cldcvr/flow-lineage

[3.1.1] - 2023-12-19

Patch Changes

  • 🔄 Updated the template rendering logic of d3's foreignObject for enhanced security against HTML injection. 🛡️

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions

58.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@datapipes-robot datapipes-robot bot added the lgtm label Dec 19, 2023
@vikas-cldcvr vikas-cldcvr merged commit 38aafbc into main Dec 19, 2023
4 of 5 checks passed
@vikas-cldcvr vikas-cldcvr deleted the vikas/FLOW-1037 branch December 19, 2023 11:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@NikhilVerma NikhilVerma NikhilVerma approved these changes

Assignees

@NikhilVerma NikhilVerma

Labels
lgtm size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vikas-cldcvr @NikhilVerma