Skip to content

Commit dceac69

Browse files
author
Juan Pablo Santos
committed
Unpack constructor before passing to signing function
1 parent b7bc8e4 commit dceac69

File tree

4 files changed

+8
-11
lines changed

4 files changed

+8
-11
lines changed

Network/Wreq/Internal.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ prepare modify opts url = do
118118
signRequest = maybe return f $ auth opts
119119
where
120120
f (AWSAuth versn key secret) = AWS.signRequest versn key secret
121-
f oauth1Credentials@(OAuth1 _ _ _ _) = OAuth1.signRequest oauth1Credentials
121+
f (OAuth1 consumerToken consumerSecret token secret) = OAuth1.signRequest consumerToken consumerSecret token secret
122122
f _ = return
123123

124124

Network/Wreq/Internal/OAuth1.hs

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,12 @@
1-
module Network.Wreq.Internal.OAuth1
2-
(
3-
signRequest
4-
) where
1+
module Network.Wreq.Internal.OAuth1 (signRequest) where
52

63
import Network.HTTP.Client (Request(..))
7-
import Network.Wreq.Internal.Types (Auth(..))
84
import Web.Authenticate.OAuth ( signOAuth, newOAuth, oauthConsumerKey
95
, oauthConsumerSecret, newCredential)
6+
import qualified Data.ByteString as S
107

11-
signRequest :: Auth -> Request -> IO Request
12-
signRequest (OAuth1 consumerToken consumerSecret token tokenSecret) requestToSign = signOAuth app creds requestToSign
8+
signRequest :: S.ByteString -> S.ByteString -> S.ByteString -> S.ByteString -> Request -> IO Request
9+
signRequest consumerToken consumerSecret token tokenSecret = signOAuth app creds
1310
where
1411
app = newOAuth { oauthConsumerKey = consumerToken, oauthConsumerSecret = consumerSecret }
1512
creds = newCredential token tokenSecret
16-
17-
signRequest _ requestToSign = return requestToSign

Network/Wreq/Internal/Types.hs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -178,6 +178,8 @@ data Auth = BasicAuth S.ByteString S.ByteString
178178
-- ^ Amazon Web Services request signing
179179
-- AWSAuthVersion key secret
180180
| OAuth1 S.ByteString S.ByteString S.ByteString S.ByteString
181+
-- ^ OAuth1 request signing
182+
-- OAuth1 consumerToken consumerSecret token secret
181183
deriving (Eq, Show, Typeable)
182184

183185
data AWSAuthVersion = AWSv4

wreq.cabal

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,9 +80,9 @@ library
8080
other-modules:
8181
Network.Wreq.Internal
8282
Network.Wreq.Internal.AWS
83-
Network.Wreq.Internal.OAuth1
8483
Network.Wreq.Internal.Lens
8584
Network.Wreq.Internal.Link
85+
Network.Wreq.Internal.OAuth1
8686
Network.Wreq.Internal.Types
8787
Network.Wreq.Lens.Machinery
8888
Network.Wreq.Lens.TH

0 commit comments

Comments
 (0)