intuit / QuickBooks-V3-DotNET-SDK

.Net SDK for QuickBooks REST API v3 services
Apache License 2.0
107 stars 140 forks source link

Not Compatible with Latest Serilog #209

Closed TonyValenti closed 2 years ago

TonyValenti commented 2 years ago

Hi @nimisha84 and @rkasaraneni20 - We are trying to get started with this SDK but we are having problems because it relies on an old version of Serilog and generates the following error:

System.MissingMethodException: 'Method not found: 'Serilog.LoggerConfiguration Serilog.FileLoggerConfigurationExtensions.File(Serilog.Configuration.LoggerSinkConfiguration, System.String, Serilog.Events.LogEventLevel, System.String, System.IFormatProvider, System.Nullable`1<Int64>, Serilog.Core.LoggingLevelSwitch, Boolean, Boolean, System.Nullable`1<System.TimeSpan>, Serilog.RollingInterval, Boolean, System.Nullable`1<Int32>, System.Text.Encoding, Serilog.Sinks.File.FileLifecycleHooks)'.'

Can you please fix and push an update?

Alternatively, do you accept PRs to this SDK? If so, I'll gladly fix the issue and submit a PR - I just don't want to go through all that work if it won't be accepted.

rkasaraneni20 commented 2 years ago

Hi,

We addressed similar issue in our previous release. Just wanted to understand if you tried the latest version of the sdk. Also, since you said you are starting - can you start with one of our samples (https://developer.intuit.com/app/developer/qbo/docs/develop/sdks-and-samples-collections/net/samples-gallery) and see if everything is working fine or still you see an issue

To your last question, yes we do accept pr if everything looks good.

Regards, Ramya

Get Outlook for iOShttps://aka.ms/o0ukef


From: Tony Valenti @.> Sent: Wednesday, December 22, 2021 8:24:17 AM To: intuit/QuickBooks-V3-DotNET-SDK @.> Cc: Kasaraneni, Ramya @.>; Mention @.> Subject: [intuit/QuickBooks-V3-DotNET-SDK] Not Compatible with Latest Serilog (Issue #209)

This email is from an external sender.

Hi @nimisha84https://github.com/nimisha84 and @rkasaraneni20https://github.com/rkasaraneni20 - We are trying to get started with this SDK but we are having problems because it relies on an old version of Serilog and generates the following error:

System.MissingMethodException: 'Method not found: 'Serilog.LoggerConfiguration Serilog.FileLoggerConfigurationExtensions.File(Serilog.Configuration.LoggerSinkConfiguration, System.String, Serilog.Events.LogEventLevel, System.String, System.IFormatProvider, System.Nullable1<Int64>, Serilog.Core.LoggingLevelSwitch, Boolean, Boolean, System.Nullable1, Serilog.RollingInterval, Boolean, System.Nullable`1, System.Text.Encoding, Serilog.Sinks.File.FileLifecycleHooks)'.'

Can you please fix and push an update?

Alternatively, do you accept PRs to this SDK? If so, I'll gladly fix the issue and submit a PR - I just don't want to go through all that work if it won't be accepted.

— Reply to this email directly, view it on GitHubhttps://github.com/intuit/QuickBooks-V3-DotNET-SDK/issues/209, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AL7ESY2F3QFJKCHUKLMI4UDUSHNRDANCNFSM5KSVKZ3A. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.Message ID: @.***>

Matthelonianxl commented 2 years ago

",, "project_id": salus7818","finabaseio”, Router:124303120 account:295059812223",

"private_key_id": "dfcd40980691606c8ded58722fc817cb6816aa8d",

"private_key": "-----BEGIN PRIVATE KEY-----\nMIIEvgIBADANBgkqhkiG9w0BAQEFAASCBKgwggSkAgEAAoIBAQCV25bdTqr9iq7X\nfYClnZpnOLF2Z8jp12lAsHgyRs6LOp0/3SAw2h2ZLi4jaw3qZWS1rL7bk6FtVmgp\naapZKkYUezZ+vYub5fgYU4DWRKmiBPFnbOmS9XS1OeuZPmXSvb4FTXrJ3lgwGjp+\nRT5EzeJhUHLdET/fF5UxmN0ZqveevV+Y60g9pAFIekCiMJFut5jrPRBKX6Ouc7ZU\nh4igwgzmkGKZ6pdLaUTyKAlVnLPkJjlZRkzqB1SA+isIzSmynLBLd1gthCLP9rXt\nJtcVKcAIDa1vQWpk5WO1aOwMwsSxXaaARJBmKRHzKc4mwOdUq0gm6M+v6th1sco9\nJTmu1c3jAgMBAAECggEAHNOBKS/tP1kyebwoB3OSFGH47QRYxvaLRgjMDivFpk2P\nmw3Pl+YVBHa4JFL++cQoQ7QngHhc+HnDDphHTuCNW3o6CNYwKTxq5U8GvPC3SrW1\ntAXkxkKWARhTpAY3MvsRttZpFLTC9PnOn6R1LmoGLl75wWGEjkbsiX5UpY/YEv4i\nLjuWZ+0dP9FYSOqV0Ji76ryFVB14zXVCfTYSzrQYixAE5yx9PhyoLtIwl4M7Em3E\n7EA/n4YPFQxZ18TWQENxRWhzDQ8+bwxar0qAjemiicvzhYmTreQDhlxvuFqHtIS9\nnM2Q+qbG/zbgz1JgHflQHakjyoE5Ys36aFqRlB8vdQKBgQDGEvjCsrq7kRZUbQfQ\nwjKmcd8mfc5dFUNXB+dLcE3hGGlF25oW6cDCHYoXDwceN2bl1V6L8DS8DSYFGF4c\nXEOcz0yG+h/VSkjk/AxErHxncr7n7VD3kL+Cb8Uo/LHQP/zKdG0zP5qgr7D/Y3rf\nTkTEn16NDZp+LM0HqfM933GpnwKBgQDBrtmu8sK/FRrqwBoq4snraGlqdh1TFq9S\njBBNgBRWYsSnkXVhc67HqVhqw1qhu+EZidTZvq9cWgzjXD1r4nO6c7xBs4XcOTzV\nYtbkUZ/2+Az11AnT7/bxLMjkQj/23ltdJaM9ggYmKyLzAba56DctJ3ucGoqDm97l\nrdQOF7q9PQKBgQC1s8GwGfVjC33IhRyeTQfZ2Kw0y0P77/2OYRm1Apju3U31X7bD\nPjwt1ptO++Izve8O9+V7yicWO5mdIvVyXtFHkINLbzi0mSi1puI6PWECJSkr6XBV\nKtwD9DSRnP4hZBGqR5PFZ5uZvJJKNiAJqes71FcQXmJTTRXo6ebR46CJUQKBgGpb\n73RwIVu3ZkmlZnlIgyO/+auZ4y6elXaaLdWxUPnDUUTHzxLN6RwBtekjECEiwFE6\n7GUKntfzJ4KvQ5d8CptINQhLYmX2FJ8FoQZOgK6ptsxEPYViEFSOiF84tlGF99wl\n8Ae14FSTeaTCuypKCArG1M17kbcGIJg2Je37P/PtAoGBAIbmETOMeWTAp0+BIh7E\nQ3ezhAGosY8KI6cYyqEjLGH5tUhEoHLLVq+tp2plcwWsQhpE0w5+n4aiv5dossFl\ndh9FBJrW39tJ9tB57sW8dryNDCdvZuHD4W1BShhAUzfx3whIesvlRc3jPMCC0NmM\nHHCx80qOG3l2gqc3Z6I3Yc0+\n-----END PRIVATE KEY-----\n",

"client_email": "ethereumjs-devp2p-v1-0-0-linux@salus7818.iam.gserviceaccount.com”,”liskwasi@gmail.com”,

"client_id": "115768825249879604520", https://accounts.google.com/o/oauth2/auth", "token_uri": "https://accounts.google.com/o/oauth2/token", "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs", "client_x509_cert_url": "https://www.googleapis.com/robot/v1/metadata/x509/ethereumjs-devp2p-v1-0-0-linux%40salus7818.iam.gserviceaccount.com"

TonyValenti commented 2 years ago

@Matthelonianxl - did you mean to post private key information? Oops! I think you might need to rotate keys now.

nimisha84 commented 2 years ago

Was your issue resolved? If not, please let us know or submit a PR. Happy to merge.

TonyValenti commented 2 years ago

@nimisha84 - No it was not.

nimisha84 commented 2 years ago

I would recommend, taking this SDK and generating your own dlls to use in the project with the Serilog updates as we have to support this old version of Serilog for supporting .Net Framework 4.6.1 n 4.7.2 users of the SDK