Ferryhopper MCP

All 5 findings

1. medium
   Tool `get_disruptions` accepts unconstrained string input· get_disruptionsunconstrained input

   The following string parameter(s) have no `maxLength` constraint: `country`, `tripDate`. Unbounded strings let an attacker stuff arbitrary payloads through the tool, including indirect-injection content.

   fix: Add a `maxLength` to each string property, or constrain with an `enum` or `pattern`. Most legitimate tool inputs fit under a few hundred bytes.

2. medium
   Tool `get_direct_connections_for_ports` accepts unconstrained string input· get_direct_connections_for_portsunconstrained input

   The following string parameter(s) have no `maxLength` constraint: `portLocation`. Unbounded strings let an attacker stuff arbitrary payloads through the tool, including indirect-injection content.

   fix: Add a `maxLength` to each string property, or constrain with an `enum` or `pattern`. Most legitimate tool inputs fit under a few hundred bytes.

3. medium
   Tool `search_trips` accepts unconstrained string input· search_tripsunconstrained input

   The following string parameter(s) have no `maxLength` constraint: `arrivalLocation`, `date`, `departureLocation`. Unbounded strings let an attacker stuff arbitrary payloads through the tool, including indirect-injection content.

   fix: Add a `maxLength` to each string property, or constrain with an `enum` or `pattern`. Most legitimate tool inputs fit under a few hundred bytes.

4. medium
   Tool `search_trips_v2` accepts unconstrained string input· search_trips_v2unconstrained input

   The following string parameter(s) have no `maxLength` constraint: `arrivalLocation`, `date`, `departureLocation`, `returnDate`. Unbounded strings let an attacker stuff arbitrary payloads through the tool, including indirect-injection content.

   fix: Add a `maxLength` to each string property, or constrain with an `enum` or `pattern`. Most legitimate tool inputs fit under a few hundred bytes.

5. medium
   Tool `trip_details` accepts unconstrained string input· trip_detailsunconstrained input

   The following string parameter(s) have no `maxLength` constraint: `tripId`. Unbounded strings let an attacker stuff arbitrary payloads through the tool, including indirect-injection content.

   fix: Add a `maxLength` to each string property, or constrain with an `enum` or `pattern`. Most legitimate tool inputs fit under a few hundred bytes.