Florian Sager

Results 16 comments of Florian Sager

I increased the compatibility of FILTER RULES in https://github.com/usrflo/yajsync/tree/master to rsync. The system tests consider yajsync-client or rsync-client to yajsync-server. Core support is available but more tests have to be...

I now manually merged my changes from 2014, 2015 and 2016 to the following branch: https://github.com/usrflo/yajsync/tree/filter2. This branch is based on your last master branch. The SystemTests are running but...

https://github.com/usrflo/yajsync/tree/filter2 now contains SystemTest with parameterized yajsync/rsync tests. This setup is a bit more complicated than the abstract test class approach. Moreover a parameter dependent skipping of tests has to...

Please re-open this issue so the filter topics gets more attention.

@perlundq : could you please review https://github.com/usrflo/yajsync/tree/filter2 and give feedback to - the implementation of parameterized system tests (for parallel yajsync and rsync testing) - the filter implementation itself It...

@perlundq : is there any chance you take a look at the linked filter implementation?

I modified https://github.com/bbeck/token-bucket to remove any dependency on https://github.com/google/guava, see [token-bucket-wo-dependencies.zip](https://github.com/perlundq/yajsync/files/672982/token-bucket-wo-dependencies.zip). Would it fit into the design of yajsync to add the resulting classes to a separate package like com.github.perlundq.yajsync.internal.util.ratelimit?...

Thanks, but I left the originally chosen coding style as it was.

@wadih, thanks for your feedback and research: in https://github.com/usrflo/registered-domain-libs/commit/9fccafa8a54c1d558fec1bcc4693a1163f2d3ed2 I simply disabled the domain label length validation and the regexp to prevent false analysis. I tend to change the processing...

I had the same error situation as described above (https://github.com/Mailtrain-org/mailtrain/issues/1075#issue-891144980) and I needed 4 changes to get this to run: 1. although in server/package.json the dependency on "posix": "^4.1.2" is...