Skip to content

docs: document IllegalArgumentException on URL setters #9707

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.

Sign up for GitHub

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

Merged
web-padawan merged 1 commit into from
Jul 9, 2026
+19 −0
Merged

docs: document IllegalArgumentException on URL setters #9707

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions ...breadcrumbs-flow/src/main/java/com/vaadin/flow/component/breadcrumbs/BreadcrumbsItem.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -234,8 +234,14 @@ public String getPath() {
* @param path
* the path to link to, or {@code null} to remove the path and
* render the item as the current page (a non-link)
* @throws IllegalArgumentException
* if {@code path} uses a scheme that is not considered safe;
* see {@link #setUnsafePath(String)} and the
* {@value InitParameters#URL_SAFE_SCHEMES} configuration
* property
*
* @see #setPath(Class)
* @see #setUnsafePath(String)
*/
public void setPath(String path) {
if (path != null && !UrlUtil.isSafeUrl(path)) {
Expand Down
7 changes: 7 additions & 0 deletions ...rent/vaadin-charts-flow/src/main/java/com/vaadin/flow/component/charts/model/Credits.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,13 @@ public String getHref() {
* The URL for the credits label.
* <p>
* Defaults to: http://www.highcharts.com
*
* @throws IllegalArgumentException
* if {@code href} uses a scheme that is not considered safe;
* see {@link #setUnsafeHref(String)} and the
* {@value InitParameters#URL_SAFE_SCHEMES} configuration
* property
* @see #setUnsafeHref(String)
*/
public void setHref(String href) {
if (href != null && !UrlUtil.isSafeUrl(href)) {
Expand Down
6 changes: 6 additions & 0 deletions ...ent/vaadin-side-nav-flow/src/main/java/com/vaadin/flow/component/sidenav/SideNavItem.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -262,8 +262,14 @@ private void removeLabelElement() {
* @param path
* The path to link to. Set to null to disable navigation for
* this item.
* @throws IllegalArgumentException
* if {@code path} uses a scheme that is not considered safe;
* see {@link #setUnsafePath(String)} and the
* {@value InitParameters#URL_SAFE_SCHEMES} configuration
* property
*
* @see SideNavItem#setPath(Class)
* @see #setUnsafePath(String)
*/
public void setPath(String path) {
if (path != null && !UrlUtil.isSafeUrl(path)) {
Expand Down
Loading