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

Receiver doesn't see EventPaymentReceivedSuccess #2461

Closed
eorituz opened this issue Sep 13, 2018 · 8 comments
Closed

Receiver doesn't see EventPaymentReceivedSuccess #2461

eorituz opened this issue Sep 13, 2018 · 8 comments

Comments

@eorituz
Copy link
Contributor

eorituz commented Sep 13, 2018

Problem Definition

When sending a sucessfull Tx between two locally running nodes (I ran the nodes in debug mode and saw the EventPaymentSentSuccess in the senders and the receivers Raiden client) the receiver doesn't store the event correctly.

Meaning that querying the payment history between S and R after the Tx:
The S response will be the expected EventPaymentSentSuccess.
While the R node does return an empty list.

System Description

Ubuntu 18.04, python3.6, Raiden version 0.8.1.dev78+g9373c616 (latest master on Tuesday 13.09)

@LefterisJP
Copy link
Contributor

Can you go into a bit more details about what you are doing?

  • What are the exact API endpoints that each node queries and with what parameters
  • What is the response?
  • What is your expected response
  • Attach logs if possible

@eorituz
Copy link
Contributor Author

eorituz commented Sep 13, 2018

@LefterisJP Sure:

Raiden Client Log files:

Sender (on Port 5009)
Receiver (on Port 5001)

Sender_address = 0x00F3fF781d1eCf923aFa54611CE6C3c9E679D1E3
Receiver_address = 0x002857f3a3BEa9DC0301D6DCf573692720f88B5a

The Payment was initialized by the API call:

echo '{ "amount": 1, "identifier": 2 }' | http POST http://localhost:5009/api/1/payments/0x0f114A1E9Db192502E7856309cc899952b3db1ED/0x002857f3a3BEa9DC0301D6DCf573692720f88B5a Content-Type:application/json

The Response was

HTTP/1.1 200 OK
Content-Length: 224
Content-Type: application/json
Date: Thu, 13 Sep 2018 11:52:39 GMT
mimetype: application/json

{
    "amount": 1,
    "identifier": 2,
    "initiator_address": "0x00F3fF781d1eCf923aFa54611CE6C3c9E679D1E3",
    "target_address": "0x002857f3a3BEa9DC0301D6DCf573692720f88B5a",
    "token_address": "0x0f114A1E9Db192502E7856309cc899952b3db1ED"
}

Payment API requests

Receivers payment history request:

http http://localhost:5001/api/1/payments/0x0f114A1E9Db192502E7856309cc899952b3db1ED/0x00F3fF781d1eCf923aFa54611CE6C3c9E679D1E3

Response:

HTTP/1.1 200 OK
Content-Length: 2
Content-Type: application/json
Date: Thu, 13 Sep 2018 13:30:18 GMT
mimetype: application/json

[]

Senders payment history request

http http://localhost:5009/api/1/payments/0x0f114A1E9Db192502E7856309cc899952b3db1ED/0x002857f3a3BEa9DC0301D6DCf573692720f88B5a

Response:

HTTP/1.1 200 OK
Content-Length: 124
Content-Type: application/json
Date: Thu, 13 Sep 2018 13:30:30 GMT
mimetype: application/json

[
    {
        "amount": 1,
        "event": "EventPaymentSentSuccess",
        "identifier": 2,
        "target": "0x002857f3a3BEa9DC0301D6DCf573692720f88B5a"
    }
]

@czepluch
Copy link
Collaborator

I can verify that this is a problem. I thought we fixed it with https://github.com/raiden-network/raiden/pull/2382/files

@czepluch
Copy link
Collaborator

czepluch commented Sep 13, 2018

And it's very weird that the EventPaymentReceivedSuccess doesn't show, since we have a test for it (https://github.com/raiden-network/raiden/pull/2395/files)

@czepluch czepluch changed the title Receiver doesn't handle EventPaymentSentSuccess correctly Receiver doesn't see EventPaymentReceivedSuccess Sep 13, 2018
@czepluch
Copy link
Collaborator

Related to #2373 that should have been fixed already

@LefterisJP
Copy link
Contributor

LefterisJP commented Sep 14, 2018

Thank you @eorituz

I will have a look further then if you also think it's still a problem @czepluch

@LefterisJP LefterisJP self-assigned this Sep 14, 2018
@LefterisJP LefterisJP added the State / Investigating For issues that are currently being looked into before labeling further label Sep 14, 2018
@czepluch
Copy link
Collaborator

@LefterisJP Yes, I can confirm that it's still a problem. I just now tried with latest master again and it is indeed a problem.

@LefterisJP
Copy link
Contributor

Ah damn I think I see the problem.

@LefterisJP LefterisJP added Severity / Minor and removed State / Investigating For issues that are currently being looked into before labeling further labels Sep 14, 2018
@LefterisJP LefterisJP added this to the Red Eyes Testnet 9 milestone Sep 14, 2018
LefterisJP added a commit to LefterisJP/raiden that referenced this issue Sep 14, 2018
LefterisJP added a commit to LefterisJP/raiden that referenced this issue Sep 14, 2018
LefterisJP added a commit to LefterisJP/raiden that referenced this issue Sep 14, 2018
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

3 participants