Project

General

Profile

Actions

Bug #13600

closed

CSP violation with WMenuItem

Added by Steven Köhler about 2 months ago. Updated 14 days ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
03/03/2025
Due date:
% Done:

100%

Estimated time:

Description

While trying out the new nonce functionality, I noticed a Content Security Policy violation. Here is an attached screenshot of the error message.

It occured with a WPopupMenu widget while clicking one of the items. I could also reproduce the issue with WNavigationBar, WMenu and WTabWidget - basically everything that uses WMenuItem, which is represented by an <a> tag. The issue itself seems to be caused by the href="javascript:void(0);" attribute. Since this is intended to do nothing, the functionality of the widget itself is not impared, but everytime an item is selected a CSP violation error is logged in the browser console.

Best regards,
Steven


Files

csp-err.PNG (34.5 KB) csp-err.PNG Steven Köhler, 03/03/2025 09:54 AM
Actions #1

Updated by Matthias Van Ceulebroeck about 2 months ago

  • Target version set to 4.11.4

Hello Steven,

that does indeed seem like something missed in #13501. I believe those were added to make tabbing over those elements work correctly. We'll look into alternatives for it.

Best,
Matthias

Actions #2

Updated by Romain Mardulyn about 2 months ago

  • Status changed from New to InProgress
  • Assignee set to Romain Mardulyn
Actions #3

Updated by Romain Mardulyn about 2 months ago

  • Status changed from InProgress to Review
  • Assignee deleted (Romain Mardulyn)
Actions #4

Updated by Matthias Van Ceulebroeck 25 days ago

  • Assignee set to Matthias Van Ceulebroeck
Actions #5

Updated by Matthias Van Ceulebroeck 16 days ago

  • Status changed from Review to Implemented @Emweb
  • Assignee changed from Matthias Van Ceulebroeck to Romain Mardulyn
  • % Done changed from 0 to 100
Actions #6

Updated by Matthias Van Ceulebroeck 14 days ago

  • Status changed from Implemented @Emweb to Implemented @Test
Actions #7

Updated by Matthias Van Ceulebroeck 14 days ago

  • Status changed from Implemented @Test to Closed
Actions

Also available in: Atom PDF