Bicycle allowed on vehicle=delivery or vehicle=forestry

[zdila]

Hello,

Why is bicycle (bike2, mtb, racingbike) allowed on roads with vehicle=delivery or vehicle=forestry access tag?

Calculated route has get_off_bike set to false.

vehicle=delivery:

https://graphhopper.com/maps/?point=48.811838%2C21.529684&point=48.795458%2C21.540992&locale=sk&elevation=true&profile=bike&use_miles=false&selected_detail=Elevation&layer=Omniscale

vehicle=forestry:

https://graphhopper.com/maps/?point=48.912984%2C21.474913&point=48.898662%2C21.511931&locale=sk&elevation=true&profile=bike&use_miles=false&selected_detail=Elevation&layer=Omniscale

[karussell]

Not sure if this is intentional or not. Will try if I can reproduce this. According to OSMRoadAccessParser.toOSMRestrictions it shouldn’t (for BIKE).

See the OSM way Way: 310852249 | OpenStreetMap

[karussell]

Yes, we currently allow access to ways with these values. The restrictions are only applied when vehicle or access is one of those

restrictedValues.add("no");
restrictedValues.add("restricted");
restrictedValues.add("military");
restrictedValues.add("emergency");
restrictedValues.add("private");

@zdila Do you think this is a bug in this specific case?

@ratrun what do you think in general? All bikes are vehicles and probably we should use the same values that we use for car?

        restrictedValues.add("agricultural");
        restrictedValues.add("forestry");
        restrictedValues.add("no");
        restrictedValues.add("restricted");
        restrictedValues.add("delivery");
        restrictedValues.add("military");
        restrictedValues.add("emergency");
        restrictedValues.add("private");

[ratrun]

@karussell, yes I agree with you that we should add thoose three values also for bikes. It is a bit strange that it took that long to detect this bug.

[karussell]

Merged your changes - thanks @ratrun !