Means to silence log "Reverse sync will not be enabled" #602
Labels
Comments
AlanCoding
added a commit
that referenced
this issue
Sep 4, 2024
Due to dynaconf implementation details, I discovered that eda-server may
set `{"URL": None}` for the `RESOURCE_SERVER` setting. This is totally
fine with me, and should be the same as not defining that setting. But
it requires this code change.
Fixes #602
This also makes the new auth class a no-op when the resource server
is not defined, reusing that same logic.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We saw these logs from the resource provider server:
This is unnecessarily alarmist. The resource provider should never consider doing reverse sync, because it does sync, but that's a forward sync!
This proposes that if
RESOURCE_SERVER_SYNC_ENABLEDis set to False, then we simply do not emit that log. That is all. This is a non-functional change. Only logs will be affected.The text was updated successfully, but these errors were encountered: