#1 2021-12-24 18:01:15

qk
Guest

xml database injector broken again

As subject.

Cheers.

#2 2021-12-24 23:27:10

Spirit
Administrator

Re: xml database injector broken again

works fine here?

#3 2021-12-25 17:19:39

qk
Guest

Re: xml database injector broken again

Works now.

Cheers.

#4 2022-02-22 08:57:14

qk
Guest

Re: xml database injector broken again

Hi,

Broken again today at lineNumber: 29205; columnNumber: 23

The xml file should be linted/checked before uploading to avoid this common problem.

Cheers.

#5 2022-02-22 11:10:09

radiatoryang
Member

Re: xml database injector broken again

My bad. I put '&' instead of '&' ... should be fixed now.

#6 2022-02-22 11:24:08

Spirit
Administrator

Re: xml database injector broken again

It's a rite of passage ;)

#7 2022-02-22 11:27:16

Spirit
Administrator

Re: xml database injector broken again

qk: It's created on demand, but yeah it should be smarter. If you know php and can manage 10+ year old spaghetti, it would be great if https://github.com/SpiritQuaddicted/Qua … torxml.php would automatically turn UTF-8 input from the db into properly encoded XML CDATA(?) stuff.
I'd batch replace all the existing HTML entities in the DB then.

#8 2022-02-22 13:47:08

qk
Guest

Re: xml database injector broken again

Hi,

Thanks @radiator for the quick fix.

@Spirit, sometimes I am afraid of touching some of my old code so I have to decline your invitation.

BTW, wouldn't it be better to run a command line tool once the generation is done? This way you/I don't have to eat such amount of old pasta and it is easy to check the return code to avoid uploading the xml to the site in case it is broken.

Cheers.

Board footer