Closed (fixed)
Project:
Drupal Security Team
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
23 Jan 2026 at 21:42 UTC
Updated:
24 Feb 2026 at 21:14 UTC
Jump to comment: Most recent
Comments
Comment #2
gregglesThis LGTM.
Comment #3
aangel commentedComment #4
gregglesThere were some changes to the orgId in the json you provided that I didn't expect. So, I tried to manually make the changes you proposed rather than using the file as attached here.
I also used 1.0 and 1.2 for the 8.x version since I think that's how they are interpreted.
Now published and I'd appreciate your review, but consider this fixed.
Comment #6
aangel commentedFor the 8.x version, that works for me. I followed the actual version structure for this module (https://www.drupal.org/project/iframeremove/releases/8.x-1.2) but the 8.x ecosystem was a mix of version styles and I think if we just go with real semver for 8 up as a convention, that's just fine.
We could chase down why Bálint's name doesn't appear properly given that accents are common:
Bálint Nagy (nagy.balint)
It appears to have happened during input (rather than a display issue at cve.org) since the json shows ""value":"Bálint Nagy (nagy.balint)"},"
I didn't see other notable differences.
Comment #7
gregglesThanks for noticing that. I fixed Bálint Nagy (nagy.balint)'s name on the CVE just now.
For 8.x-1.2 versions, I believe they show as 8.x-1.2 in the drupal.org website, but then 1.2 in the perspective of composer. Notice how the command on that release page to install 8.x-1.2 with composer is
composer require 'drupal/iframeremove:^1.2'