changes for payu integration

[buoyant]

1. Getting merchant_id and secret_key from app environment.
2. Replaced productinfo with secret_key in generate_checksum method of helper class.
3. Sending params to notification on response.

Note: Tested with test credentials.
Any questions mail me on sandeep.kumar@payu.in

[louiskearns]

@melari can you take a look?

---

Louis

On Tue, Aug 20, 2013 at 11:33 AM, Sandeep Kumar notifications@github.comwrote:

1. Getting merchant_id and secret_key from app environment.
2. Replaced productinfo with secret_key in generate_checksum method of
   helper class.
3. Sending params to notification on response.

Note: Tested with test credentials.

Any questions mail me on sandeep.kumar@payu.in

You can merge this Pull Request by running

git pull https://github.com/buoyant/active_merchant payu

Or view, comment on, or merge it at:

#811
Commit Summary

• changes for payu integration

File Changes

• M lib/active_merchant/billing/integrations/payu_in.rbhttps://github.com/changes for payu integration #811/files#diff-0(2)
• M lib/active_merchant/billing/integrations/payu_in/helper.rbhttps://github.com/changes for payu integration #811/files#diff-1(2)
• M lib/active_merchant/billing/integrations/payu_in/notification.rbhttps://github.com/changes for payu integration #811/files#diff-2(5)

Patch Links:

• https://github.com/Shopify/active_merchant/pull/811.patch
• https://github.com/Shopify/active_merchant/pull/811.diff

[jduff]

These cannot be set this way, it doesn't work in a multi client environment. The credentials and secrets have to be passed in to the initialize method.

[buoyant]

OK. but since payu_in.rb is a module can I create a new method and pass variables to it.

def self.new(options)
@merchant_id = options[:merchant_id]
@secret_key = options[:secret_key]
end

[jduff]

No, needs to be passed into the notification, helper and return themselves. Ipay88 is one I fixed to do this correctly and would be a good example.

[buoyant]

Important thing we don't want salt/secret_key to put in form as hidden field.

[buoyant]

Any alternative for hidden field.

[ntalbott]

@jduff have you had a chance to review this since @buoyant's last change?

[jduff]

It doesn't look like all the changes required were made.

[ntalbott]

ActiveMerchant integrations have been extracted into https://github.com/Shopify/offsite_payments. This PR has been moved (via reference) to the new repo - please follow up on it there.