dstucrypt / agent

47 stars 31 forks source link

repair sending from client to remote box over TCP #49

Closed zavla closed 1 year ago

zavla commented 1 year ago

Добрий день! Добрий день. Не працювала передача з клієнта в RemoteBox файлів по TCP. Виправив. Тако ж не було передбачено роботу в RemoteBox з двома файлами від клієнта при --verify: наприклад перший файл це p7s, а другій файл це оригінальний файл. Це коли підпис в окремому файлі. Виправив. Не дуже впевнений що зроблено вірно. Робив під Windows. Гляньте, будь ласка.

muromec commented 1 year ago

Кросиво, дякую. Можна трішечки поміняти і я тоді замержу

muromec commented 1 year ago

Кросиво. Розбийте бумласочка на два патчі і заберіть ca bundle і vscode і най буде

zavla commented 1 year ago

Ага, ось зараз розумію, не правильно робити коміти де в одному коміті навалено декілька логічних змін. Ви мабудь про це коли кажете "зробіть бумласочка двома патчами". Патч то коміт? Я поспішив, винуват, накомітів великими порціями. Що тепер робити ?

muromec commented 1 year ago

Ви мабудь про це коли кажете "зробіть бумласочка двома патчами".

все так

Я поспішив, винуват, накомітів великими порціями.

нема проблеми, просто зробіть окремий бранч з _data2, а в цьому нехай буде тільки TCP.

В цьому бранчі робите git checkout -b новий_бранч_data2, потім повертаєтесь до цього і видаляєете все що не TCP. Я замержу цей бранч з TCP, потім відкриєте новий PR з тим що лишилося.

muromec commented 1 year ago

окей, оцю останню штуку приберіть і я натискаю велику зелену кнопку

zavla commented 1 year ago

Вже маю на github підготовлену другу гілку ту що виправляє не роботу --verify режиму з двома файлами від клієнта.

muromec commented 1 year ago

Красно дякую!