Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

get DESTFN=NONE::TIME to work... I think. #1323

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

petersilva
Copy link
Contributor

fixing @andreleblanc11 's problem... maybe I think it will work, but did not test. It at least causes no breakage.

@petersilva petersilva marked this pull request as ready for review December 4, 2024 22:19
Copy link

github-actions bot commented Dec 4, 2024

Test Results

238 tests   235 ✅  1m 32s ⏱️
  1 suites    1 💤
  1 files      2 ❌

For more details on these failures, see this check.

Results for commit 3ebb5a0.

@andreleblanc11
Copy link
Member

andreleblanc11 commented Dec 5, 2024

I just tested it. Looks like it works!

I had filename NONE::TIME in my config.

2024-12-05 14:07:46,716 [DEBUG] sarracenia.flow filter  urlToMatch: file:/net/local/home/leblanca/test/myfile1
2024-12-05 14:07:46,716 [DEBUG] sarracenia.flow filter end len(incoming)=1, rejected=0
2024-12-05 14:07:46,716 [INFO] sarracenia.flowcb.log after_accept accepted: (lag: 95.01 )  exchange: xs_tfeed subtopic: net.local.home.leblanca.test a file
with baseUrl: file: relPath: /net/local/home/leblanca/test/myfile1 id: 1B2M2Y8 size: 0
2024-12-05 14:07:46,716 [DEBUG] sarracenia.flow filter B filtered incoming: 1, ok: 0 (directories: 0), rejected: 0, failed: 0 stop_requested: False have_vip
: ['AnyAddressIsFine']
2024-12-05 14:07:46,716 [DEBUG] sarracenia.flow send file_transport sendTo: file://sarra@localhost/
2024-12-05 14:07:46,716 [DEBUG] sarracenia.flow send file_transport send /net/local/home/leblanca/test myfile1::20241205140611

@petersilva petersilva merged commit 622b79f into development Dec 5, 2024
30 of 59 checks passed
@petersilva petersilva deleted the NONE_doublecolon_TIME branch December 13, 2024 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants