M Hida

> Why did you close it? sorry i dont know why it closed, i just refresh browser and this PR closed. reopen it.

Ok next time. Cz little busy at this time. And note that this currently validate only signer certificate, not intermediate or multiple certificate path (no problem, just make validation function...

> What I do not like from these changes is that those scripts that are included here are supposed to be simple examples. umm I think this still simple, we...

> > I'm also interested if we can define signature max length before signing > > Those optimizations would be better in a separate PR i have updated, but already...

@erikn69 I think with single merged script we can add or pass tsa. just set tsa url parameter to empty to bypass it? cmiiw..

> > [merged pdfsignltv.php & pdfsigntsa.php](https://github.com/dealfonso/sapp/pull/70/commits/6b263ad113bf0c8fa97040db09fb801c9bc43355) > > I actually went to the trouble of separating it for a good reason. They can be used separately, That's why I created...

> `?: "http://timestamp.digicert.com";` you removed the default value, but on the text shows `digicert` as default > > [erikn69@ac6ae19](https://github.com/erikn69/sapp/commit/ac6ae194672601197027574dcea7ba0f6c24a642) I don't want to remove remove default value, but how to...

> > but how to bypass timestamping without removing that? any idea? > > That's why I created 2 separate scripts, but let's leave it like this, this does not...

@erikn69 > Fully working, but LTV is pending > > Also, the code is from [#8 (comment)](https://github.com/dealfonso/sapp/issues/8#issuecomment-1059107230) and [#8 (comment)](https://github.com/dealfonso/sapp/issues/8#issuecomment-1470039444), and there are probably shorter and better ways to do...

implement in [https://github.com/dealfonso/sapp/pull/70](url) check it out.