Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Orion-LD crashes with this payload #109

Closed
jmcanterafonseca opened this issue Jun 11, 2019 · 4 comments
Closed

Orion-LD crashes with this payload #109

jmcanterafonseca opened this issue Jun 11, 2019 · 4 comments

Comments

@jmcanterafonseca
Copy link

POST /ngsi-ld/v1/entities/

{
    "id": "urn:ngsi-ld:ParkingSpot:santander:daoiz_velarde_1_5:3",
    "type": "ParkingSpot",
    "status": {
        "type": "Property",
        "value": "free",
        "observedAt": "2018-09-21T12:00:00Z"
    },
    "category": {
        "type": "Property",
        "value": [
            "onstreet"
        ]
    },
    "refParkingSite": {
        "type": "Relationship",
        "object": "urn:ngsi-ld:ParkingSite:santander:daoiz_velarde_1_5"
    },
    "name": {
        "type": "Property",
        "value": "A-13"
    },
    "location": {
        "type": "GeoProperty",
        "value": {
            "type": "Point",
            "coordinates": [
                -3.80356167695194,
                43.46296641666926
            ]
        }
    },
    "@context": [
        "https://uri.etsi.org/ngsi-ld/v1/ngsi-ld-core-context.jsonld",
        "https://schema.lab.fiware.org/ld/context"
    ]
}
@kzangeli
Copy link
Collaborator

kzangeli commented Jul 1, 2019

I made a functest but didn't get any crash. Worked just fine ...
Try the test again with an up-to-date broker?

@kzangeli
Copy link
Collaborator

kzangeli commented Jul 1, 2019

UPDATE:
Found a few strange things when compiling in my CentOS VM.
Stuff that could provoke a crash.
It works just fine in Ubuntu but may very well crash in CentOS.
PR on its way

@kzangeli
Copy link
Collaborator

kzangeli commented Jul 1, 2019

The PR "on the way" has been merged: #121
Fixed two errors detected by the CentOS compiler, unfortunately the compiler in Ubuntu didn't catch these two errors.
One of them could provoke a crash ...

@kzangeli
Copy link
Collaborator

kzangeli commented Jul 3, 2019

Another PR, an older one I had "forgotten", with a functest making sure this issue is a non-issue has been merged as well. With all tests 100% OK in travis: #120

@kzangeli kzangeli closed this as completed Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants