search

jvermillard / leshan

OMA Lightweight M2M java implementation - LWM2M
40 stars 25 forks source link

fixed CANCEL observe issue clientServlet.java #53

Closed alseddnm closed 9 years ago

alseddnm commented 9 years ago

fixed CANCEL observe issue.. It is a bug in client servlet line 285 -> added forward slash before observe before: String target = StringUtils.substringsBetween(req.getPathInfo(), clientEndpoint, "observe")[0]; after: String target = StringUtils.substringsBetween(req.getPathInfo(), clientEndpoint, "/observe")[0];

TARGET always = /objectId/objectInstance/resourceId/ ----- in ObservationRegistryImpl.java line 118, the key in the map does't contain the forward slash at the end..

I changed doPost also line 205 clientServlet.java.

alseddnm commented 9 years ago

fixed CANCEL observe issue..

jschloman commented 9 years ago

@alseddnm : is this branch going to be improved per the requests so we can finally merge this PR?

sbernard31 commented 9 years ago

Thx @alseddnm, I integrate your pull-request. When you mess a commit (like imports in this PR), you can fix it by using a commit --amend and a push --force on your branch. A good way to keep a clear git history.