authinfo field should be a cdata ?

metaregistrar / php-epp-client

Object-oriented PHP EPP Client

https://www.metaregistrar.com/docs/

MIT License

210 stars 154 forks source link

authinfo field should be a cdata ? #231

Closed f4810 closed 4 years ago

f4810 commented 4 years ago

Hi, I'm having trouble to send this authinfo: &&6LD"6s There is not way to get a valid XML. What can I do?

metaregistrar commented 4 years ago

Hello,

I checked and found that the login section already used CDATA for passwords, but the domain:create, domain:info and domain:update parts did not. I have now updated the code so that the domain functions also use CDATA sections for their passwords. By default, the use of CDATA is ON, but if a registry does not support it, you can switch it off with an eppRequest->setUseCdata(false);

michael-gilch commented 3 years ago

Hi, Can the useCdata() switch also be implemented for domain:transfer? Class eppTransferRequest uses hard-coded CDATA sections for transfer requests. However, all the other transfer operations (e.g. query) don't use CDATAs at all.

metaregistrar commented 3 years ago

Haha, your are right. The CDATA option was added to the login request when I encountered a registry with a password that contained < and > characters.

The CDATA option now also has been added to the eppTransferRequest and metaregEppTransferRequest