Esri / data-assistant

ArcGIS Pro Add-in that assists in emergency management, local government and state government data aggregation workflows.
Apache License 2.0
22 stars 8 forks source link

Cannot load data to a Feature Service #164

Closed MikeMillerGIS closed 7 years ago

MikeMillerGIS commented 7 years ago

Using secure feature service to my org, I am unable to load the source to target file

image

MikeMillerGIS commented 7 years ago

Looks like if I change the xml file to http: from https it loads, are you checking for only http://?

MikeMillerGIS commented 7 years ago

fix in https://github.com/Esri/data-assistant/commit/db403e620ff241da9b9e32a0056943172269cb4a

SteveGrise commented 7 years ago

yes, that's a weakness in the code.

On Wed, Apr 26, 2017 at 11:02 AM, Michael Miller notifications@github.com wrote:

Looks like if I change the xml file to http: from https it loads, are you checking for only http://?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/Esri/data-assistant/issues/164#issuecomment-297437083, or mute the thread https://github.com/notifications/unsubscribe-auth/AEh3BOt57etEfJW3B9xo8exLNOxdjRdXks5rz1x2gaJpZM4NI_ho .

MikeMillerGIS commented 7 years ago

are you saying even with the checks for https, we cannot load data to a secure service?

MikeMillerGIS commented 7 years ago

looks like I missed a few places, adding them now

MikeMillerGIS commented 7 years ago

i just noticed, that error is in the .net side, not the python

SteveGrise commented 7 years ago

yeah, I will need to check both though, there are a few parallel locations. I could probably get away with just checking for startswith('http') at this point because of the way the code works, but I like the extra check.

MikeMillerGIS commented 7 years ago

all fixed