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.

Sign up for GitHub

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

Jump to bottom

Handle EPP contact:transfer #1363

Merged
merged 1 commit into from
Oct 17, 2019
Merged

Handle EPP contact:transfer #1363

merged 1 commit into from
Oct 17, 2019

Conversation

artur-intech
Copy link
Contributor

@artur-intech artur-intech commented Oct 14, 2019 •
edited
Loading

A stub until #676 is done.

@artur-intech artur-intech force-pushed the handle-epp-contact-transfer branch 2 times, most recently from c3c49cb to 799bc1c Compare October 15, 2019 11:19
@vohmar
Copy link
Contributor

vohmar commented Oct 16, 2019

test request:

<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<epp xmlns="https://epp.tld.ee/schema/epp-ee-1.0.xsd">
  <command>
    <transfer op="request">
      <contact:transfer
       xmlns:domain="https://epp.tld.ee/schema/contact-ee-1.1.xsd">
        <contact:id>CID:EEDIRECT:275</contact:id>
      </contact:transfer>
    </transfer>
    <clTRID>38208122210-1571227869</clTRID>
  </command>
</epp>

response is malformed exception
image

@vohmar vohmar assigned artur-intech and unassigned vohmar Oct 16, 2019
@artur-intech
Copy link
Contributor Author

artur-intech commented Oct 16, 2019 •
edited
Loading

@vohmar Can you attach full output? exception caught message just duplicates your previous statement and gives me zero information.

@artur-intech artur-intech force-pushed the handle-epp-contact-transfer branch from 799bc1c to 5f23661 Compare October 16, 2019 14:17
@artur-intech artur-intech force-pushed the handle-epp-contact-transfer branch from 5f23661 to 397136e Compare October 16, 2019 14:18
@artur-intech
Copy link
Contributor Author

artur-intech commented Oct 16, 2019 •
edited
Loading

It took me two hours of debugging to realize, that the request you used is invalid.
<contact:transfer xmlns:domain part should be <contact:transfer xmlns:contact. I will create a separate ticket to give more information in case request XML is invalid.

@artur-intech artur-intech assigned vohmar and unassigned artur-intech Oct 16, 2019
@vohmar
Copy link
Contributor

vohmar commented Oct 17, 2019

my bad, sorry! But at least it revealed another bug.

@vohmar vohmar merged commit 8b3b8a3 into master Oct 17, 2019
@vohmar vohmar assigned artur-intech and unassigned vohmar Oct 17, 2019
@artur-intech artur-intech mentioned this pull request Oct 17, 2019
Open
@artur-intech artur-intech deleted the handle-epp-contact-transfer branch October 17, 2019 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@artur-intech artur-intech

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@artur-intech @vohmar