Missing VERSION error (201) from test API

krisztian-pifko commented 5 years ago

I am trying to integrate the SDK, but a basic risk query returns a 201 error. As far as I see, the SDK sends a version parameter. Below is my debug output from the client:

2019-04-24 13:13:24,400 [prezi/kount.request] DEBUG 148:set_param SDK_VERSION = Sdk-Ris-Python-0695-201904241313
2019-04-24 13:13:24,400 [prezi/kount.request] DEBUG 46:__init__ Inquiry: {'SDK_VERSION': 'Sdk-Ris-Python-0695-201904241313', 'FRMT': 'JSON', 'ANID': '', 'SDK': 'CUST'}
2019-04-24 13:13:24,400 [prezi/kount.request] DEBUG 148:set_param MERC = ******
2019-04-24 13:13:24,400 [prezi/kount.request] DEBUG 148:set_param MODE = Q
2019-04-24 13:13:24,400 [prezi/kount.request] DEBUG 148:set_param MACK = Y
2019-04-24 13:13:24,400 [prezi/kount.request] DEBUG 148:set_param SITE = PREZI
2019-04-24 13:13:24,401 [prezi/kount.request] DEBUG 148:set_param UNIQ = 7bdccfb3305011e78744784f43698186
2019-04-24 13:13:24,401 [prezi/kount.request] DEBUG 148:set_param IPAD = 80.98.71.187
2019-04-24 13:13:24,401 [prezi/kount.request] DEBUG 148:set_param NAME = hsdgfjhsgdjhfg skjdfhkjsdhfkjshf
2019-04-24 13:13:24,401 [prezi/kount.request] DEBUG 148:set_param EMAL = sldfjkjsdhfkjshdfkjhsdfkjh@kjsdhfkhsfdkjhskjfh.ff
2019-04-24 13:13:24,401 [prezi/kount.request] DEBUG 148:set_param CURR = EUR
2019-04-24 13:13:24,401 [prezi/kount.request] DEBUG 148:set_param TOTL = 228
2019-04-24 13:13:24,407 [prezi/kount.client] DEBUG 53:__init__ url - https://risk.test.kount.net, len_key - 213
2019-04-24 13:13:24,407 [prezi/kount.client] DEBUG 71:_execute IPAD=80.98.71.187
2019-04-24 13:13:24,407 [prezi/kount.client] DEBUG 71:_execute UNIQ=7bdccfb3305011e78744784f43698186
2019-04-24 13:13:24,407 [prezi/kount.client] DEBUG 71:_execute CURR=EUR
2019-04-24 13:13:24,407 [prezi/kount.client] DEBUG 71:_execute NAME=hsdgfjhsgdjhfg skjdfhkjsdhfkjshf
2019-04-24 13:13:24,407 [prezi/kount.client] DEBUG 71:_execute ANID=
2019-04-24 13:13:24,407 [prezi/kount.client] DEBUG 71:_execute SITE=*****
2019-04-24 13:13:24,408 [prezi/kount.client] DEBUG 71:_execute TOTL=228
2019-04-24 13:13:24,408 [prezi/kount.client] DEBUG 71:_execute SDK_VERSION=Sdk-Ris-Python-0695-201904241313
2019-04-24 13:13:24,408 [prezi/kount.client] DEBUG 71:_execute MODE=Q
2019-04-24 13:13:24,408 [prezi/kount.client] DEBUG 71:_execute FRMT=JSON
2019-04-24 13:13:24,408 [prezi/kount.client] DEBUG 71:_execute MERC=******
2019-04-24 13:13:24,408 [prezi/kount.client] DEBUG 71:_execute MACK=Y
2019-04-24 13:13:24,408 [prezi/kount.client] DEBUG 71:_execute EMAL=sldfjkjsdhfkjshdfkjhsdfkjh@kjsdhfkhsfdkjhskjfh.ff
2019-04-24 13:13:24,408 [prezi/kount.client] DEBUG 71:_execute SDK=CUST
2019-04-24 13:13:24,409 [prezi/kount.request] DEBUG 86:ris_validator empty value for ANID
2019-04-24 13:13:24,410 [prezi/kount.request] DEBUG 72:ris_validator missing_in_xml = SDK_VERSION
2019-04-24 13:13:24,411 [prezi/kount.request] DEBUG 72:ris_validator missing_in_xml = SDK
2019-04-24 13:13:24,411 [prezi/kount.request] DEBUG 155:ris_validator errors = [], missing_in_xml = ['SDK_VERSION', 'SDK'], empty = ['ANID']
2019-04-24 13:13:24,411 [prezi/kount.client] DEBUG 84:_execute url=https://risk.test.kount.net, headers={u'X-Kount-Api-Key': '****'}, params={'IPAD': '80.98.71.187', 'UNIQ': '7bdccfb3305011e78744784f43698186', 'CURR': u'EUR', 'NAME': 'hsdgfjhsgdjhfg skjdfhkjsdhfkjshf', 'ANID': '', 'SITE': '*****', 'TOTL': 228, 'SDK_VERSION': 'Sdk-Ris-Python-0695-201904241313', 'MODE': 'Q', 'FRMT': u'JSON', 'MERC': '******', 'MACK': 'Y', 'EMAL': u'sldfjkjsdhfkjshdfkjhsdfkjh@kjsdhfkhsfdkjhskjfh.ff', 'SDK': 'CUST'}
2019-04-24 13:13:24,542 [prezi/urllib3.connectionpool] DEBUG 824:_new_conn Starting new HTTPS connection (1): risk.test.kount.net
2019-04-24 13:13:25,176 [prezi/urllib3.connectionpool] DEBUG 396:_make_request https://risk.test.kount.net:443 "POST / HTTP/1.1" 200 25
2019-04-24 13:13:25,179 [prezi/kount.client] DEBUG 106:_execute MERC = ******, SESS = None, SDK ELAPSED = 0.634721 ms.
2019-04-24 13:13:25,179 [prezi/kount.response] DEBUG 119:__init__ RIS response init = {u'ERRO': u'201', u'MODE': u'E'}

gallilama commented 5 years ago

Hello,

I wanted to respond to your issue. I don't have a final answer because I'm not very familiar with the Python RIS SDK, this will take some time to explore. I do have some context that may (or may not!) be helpful :-)

As you noted, the SDK is setting an SDK_VERSION parameter, e.g. 'SDK_VERSION': 'Sdk-Ris-Python-0695-201904241313'

The RIS error, 201 (which means missing version) is related to a different request parameter, VERS -- This represents the version of the RIS Inquiry. This really drives the behavior, input and output of the Risk Inquiry Service. SDK_VERSION is more metadata about who's using what SDK platform at which version, etc.

Based on the example you posted, I can tell you that RIS is expecting this input: VERS=0695

What I can't answer is where / how / why is the SDK setting VERS to the same string as SDK_VERSION. You should be able to specify the value of VERS using src/kount/request

    def set_version(self, version):
        """Set the version number.
        Args: version - The SDK version  // <-- Yes, this is misleading and should be clarified! -- gallilama
        """
        self.set_param("VERS", version)

If you need a quick workaround, try setting VERS in your test client. I will carve out some time for proper root-cause analysis (Python isn't my strength so it may take longer than usual). Hope this helps.

Cheers

krisztian-pifko commented 5 years ago

@gallilama Thank you very much for the swift response! I will give the workaround a go. If VERS is just SDKConfig.SDK_VERSION, shouldn't that just automatically be set in the client?

sanjeevitt commented 5 years ago

Hello, Based on the example you posted, I can see there is some ris_validator errors = [], missing_in_xml = ['SDK_VERSION', 'SDK'], empty = ['ANID'] Here it’s showing, SDK_VERSION, SDK metadata is missing from xml file.

For that, you have to add this metadata in your kount/resource/validate.xml

`

    <max_length>20</max_length>
</param>
<param name='SDK_VERSION'>
    <max_length>30</max_length>
</param>

`

And for empty = [‘ANID’] - Automatic Number Identification (ANI) submitted with order. If the ANI cannot be determined, merchant must pass 0123456789 as the ANID. This field is only valid for MODE=P RIS submissions.

Hope it will be helpful for you.

xwchen commented 5 years ago

@sanjeevitt I'm still seeing the same error after this PR https://github.com/Kount/kount-ris-python-sdk/pull/19

Kount / kount-ris-python-sdk

Missing VERSION error (201) from test API #17