Update changelog and version

CHANGELOG.md

@@ -1,4 +1,4 @@
-# 5.9.0 RC3
+# 5.9.0
   - Changes from 5.8:
     - Algorithm:
       - Multi-Level Dijkstra:
@@ -12,6 +12,7 @@
       - `result.exits` allows you to set a way's exit numbers and names, see [`junction:ref`](http://wiki.openstreetmap.org/wiki/Proposed_features/junction_details)
       - `ExtractionWay` now as new property `forward_classes` and `backward_classes` that can set in the `way_function`.
          The maximum number of classes is 8.
+      - We now respect the `construction` tag. If the `construction` tag value is not on our whitelist (`minor`, `widening`, `no`) we will exclude the road.
     - Node.js Bindings:
       - Exposes `alternatives=Number` parameter overload in addition to the boolean flag
       - Expose `EngineConfig` options in the node bindings

features/car/construction.feature

@@ -14,3 +14,4 @@ Feature: Car - all construction tags the OpenStreetMap community could think of
             | primary      |              |     yes  |       |
             | primary      |           no |          | x     |
             | primary      |     widening |          | x     |
+            | primary      |        minor |          | x     |

package.json

@@ -1,6 +1,6 @@
 {
   "name": "osrm",
-  "version": "5.9.0-rc.3",
+  "version": "5.9.0",
   "private": false,
   "description": "The Open Source Routing Machine is a high performance routing engine written in C++14 designed to run on OpenStreetMap data.",
   "dependencies": {

profiles/bicycle.lua

@@ -72,6 +72,12 @@ local profile = {
 
   restricted_highway_whitelist = Set { },
 
+  construction_whitelist = Set {
+    'no',
+    'widening',
+    'minor',
+  },
+
   access_tags_hierarchy = Sequence {
     'bicycle',
     'vehicle',

profiles/car.lua

@@ -94,6 +94,13 @@ local profile = {
     'access'
   },
 
+  construction_whitelist = Set {
+    'no',
+    'widening',
+    'minor',
+  },
+
+
   service_tag_forbidden = Set {
     'emergency_access'
   },

profiles/foot.lua

@@ -61,6 +61,8 @@ local profile = {
 
   restricted_highway_whitelist = Set { },
 
+  construction_whitelist = Set {},
+
   access_tags_hierarchy = Sequence {
     'foot',
     'access'

profiles/lib/handlers.lua

@@ -520,7 +520,7 @@ function Handlers.handle_blocked_ways(way,result,data,profile)
     local construction = way:get_value_by_key('construction')
 
     -- Of course there are negative tags to handle, too
-    if construction and construction ~= 'no' and construction ~= 'widening' then
+    if construction and not profile.construction_whitelist[construction] then
       return false
     end
   end