{ "ignored_warnings": [ { "warning_type": "Cross-Site Scripting", "warning_code": 2, "fingerprint": "1b805585567775589825c0eda58cb84c074fc760d0a7afb101c023a51427f2b5", "check_name": "CrossSiteScripting", "message": "Unescaped model attribute", "file": "app/views/users/dossiers/_merci.html.haml", "line": 26, "link": "https://brakemanscanner.org/docs/warning_types/cross_site_scripting", "code": "current_user.dossiers.includes(:procedure).find(params[:id]).procedure.monavis_embed", "render_path": [ { "type": "controller", "class": "Users::DossiersController", "method": "merci", "line": 291, "file": "app/controllers/users/dossiers_controller.rb", "rendered": { "name": "users/dossiers/merci", "file": "app/views/users/dossiers/merci.html.haml" } }, { "type": "template", "name": "users/dossiers/merci", "line": 6, "file": "app/views/users/dossiers/merci.html.haml", "rendered": { "name": "users/dossiers/_merci", "file": "app/views/users/dossiers/_merci.html.haml" } } ], "location": { "type": "template", "template": "users/dossiers/_merci" }, "user_input": "current_user.dossiers.includes(:procedure)", "confidence": "Weak", "cwe_id": [ 79 ], "note": "" }, { "warning_type": "Cross-Site Scripting", "warning_code": 2, "fingerprint": "42099f4550a8377f455e830e8ab645cecd5806248481c5c646b4e17548c3cb07", "check_name": "CrossSiteScripting", "message": "Unescaped model attribute", "file": "app/views/france_connect/particulier/merge.html.haml", "line": 6, "link": "https://brakemanscanner.org/docs/warning_types/cross_site_scripting", "code": "t(\".subtitle\", :email => sanitize(FranceConnectInformation.find_by(:merge_token => merge_token_params).email_france_connect), :application_name => (APPLICATION_NAME))", "render_path": [ { "type": "controller", "class": "FranceConnect::ParticulierController", "method": "merge", "line": 47, "file": "app/controllers/france_connect/particulier_controller.rb", "rendered": { "name": "france_connect/particulier/merge", "file": "app/views/france_connect/particulier/merge.html.haml" } } ], "location": { "type": "template", "template": "france_connect/particulier/merge" }, "user_input": "FranceConnectInformation.find_by(:merge_token => merge_token_params).email_france_connect", "confidence": "Weak", "cwe_id": [ 79 ], "note": "explicitely sanitized even if we are using html_safe" }, { "warning_type": "SQL Injection", "warning_code": 0, "fingerprint": "737aa4f7931ece068cce98d7cc66057a1ec81b9be43e469c3569ff1be91bbf09", "check_name": "SQL", "message": "Possible SQL injection", "file": "app/graphql/connections/cursor_connection.rb", "line": 66, "link": "https://brakemanscanner.org/docs/warning_types/sql_injection/", "code": "items.order(order_column => ((:desc or :asc)), :id => ((:desc or :asc))).limit(limit).where(\"(#{order_table}.#{order_column}, #{order_table}.id) < (?, ?)\", timestamp, id)", "render_path": null, "location": { "type": "method", "class": "Connections::CursorConnection", "method": "resolve_nodes" }, "user_input": "order_table", "confidence": "Weak", "cwe_id": [ 89 ], "note": "" }, { "warning_type": "SQL Injection", "warning_code": 0, "fingerprint": "a94939cb1e551341f443c6414634816e335bbfb03f0836ebd8b3ad8564d7f343", "check_name": "SQL", "message": "Possible SQL injection", "file": "app/graphql/connections/cursor_connection.rb", "line": 69, "link": "https://brakemanscanner.org/docs/warning_types/sql_injection/", "code": "items.order(order_column => ((:desc or :asc)), :id => ((:desc or :asc))).limit(limit).where(\"(#{order_table}.#{order_column}, #{order_table}.id) > (?, ?)\", timestamp, id)", "render_path": null, "location": { "type": "method", "class": "Connections::CursorConnection", "method": "resolve_nodes" }, "user_input": "order_table", "confidence": "Weak", "cwe_id": [ 89 ], "note": "" }, { "warning_type": "SQL Injection", "warning_code": 0, "fingerprint": "bd1df30f95135357b646e21a03d95498874faffa32e3804fc643e9b6b957ee14", "check_name": "SQL", "message": "Possible SQL injection", "file": "app/models/concerns/dossier_filtering_concern.rb", "line": 32, "link": "https://brakemanscanner.org/docs/warning_types/sql_injection/", "code": "where(\"#{values.count} OR #{\"(#{ProcedurePresentation.sanitized_column(table, column)} ILIKE ?)\"}\", *values.map do\n \"%#{value}%\"\n end)", "render_path": null, "location": { "type": "method", "class": "DossierFilteringConcern", "method": null }, "user_input": "values.count", "confidence": "Medium", "cwe_id": [ 89 ], "note": "The table and column are escaped, which should make this safe" } ], "updated": "2023-08-28 12:16:04 +0200", "brakeman_version": "5.4.1" }