[withSentryRouting] match param types are made optional

[IanVS]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases
• Provide a link to the affected event from your Sentry account

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• other: @sentry/react

Version:

6.9.0

Description

I am just getting started with sentry in my typescript react project, and I found that I start to get type errors when I wrap my react-router Route components with Sentry.withSentryRouting.

Example:
https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAJQKYEMDG8BmUIjgcilQ3wFgAoUSWOAbwQgFcYkBfObXAo9GAWhzMkUPgBNc+ANwUq0eACo4KAM5wAykgB2MKAE8OOPACIAAsq07dAeh4Yj08hTQRNy+Bu16GQuAF51FnoAdADuwDAAFh6W3jDAmgDmABSxSACUDhSYjJoYwC5wAIJgYElpdBRwcAA8AHyVVTWpDY1wYCiRvkYozBHQylYAXD2R0ACSokYtjUSaosK+tEmCLAAKOGDK5b61FeStrUQwjFCaNYW90EqXUBOLK0jrEJtBIB1oEUHtUCggykEjPq3UTsKy1BwHOCsVjTMHTarRLxMFjTKrtTrdG4DYY3CZTfYHWbzKCLZbIx4bLZ+Xa0VEzJDHU7nG7XUbA+7kp4vN4wD5fFA-P4A3EguBgiEHaGw+oE6pw8gwxzkbK5OIFC5spK0QHjEGDOg64H6txQeIJVjlWkEo4nM7VUTAAButW1ItYcodzoorCAA

[AbhiPrasad]

Thanks for reporting - we will fix!

[AbhiPrasad]

This will be fixed in the next release of the SDK. For now, you can cast the type as a temp workaround. routeProps.match.params.authorId as string. Thanks again for the report and repro!

Typescript Playground Link showing how the fix works

https://www.typescriptlang.org/play?ts=4.3.5#code/JYWwDg9gTgLgBAJQKYEMDG8BmUIjgcilQ3wFgAoUSWOAbwQgFcYkBfObXAo9GAWhzMkUPgBNc+ANwUq0eLTgAVKCgB2AZ17AIquO0558AAXVJVMKAE8A9DEtgk6qTPBy4ACwjB1MAHI7kXgBlGBQYYDR1DhxDT29+VR0BYn4fMIinaXIKABskeC0ANyRlNU0MbVUALiUVDS0dLIprACo4RxzgczFvFAAjPLgjOwd1NChgMH4OrphrRL4kAA8wTrRgfjVLOBbrCmXqLEZVCp04AHcN9yCzC0sGZi6AcwAeAAV2pZZVUSjkNGgohePgmqieABo4FsAHyQhCfb6-RApAB0AGFcJBVLdFPYkO9odCABQPFg1BAASnJdAocDgAI08ABrmx5gAIt5VihLL4UCAkHAALxwElMFhQqJbCko0ScnLc3n8uAAH2VItJApQktUlmlqj5SCa5DpDJ8cAA6iowA5RBryaiAGJoglCkVgHBgdQ1N4UoXQmnGulwYCYEVFEp1crhM4AMhjcHdEE9cDjCY96hRzLAQlEAFkwmh3Cn44nPZnMTn8zBCyjvABRJa8X20WlBunh0r1U6qFGmPwGomljNZysF9wosBhdwUrJt1gUVt06zWIYwdR8YBPRJEOA3GDhMFwEDQAXqBzrTARZG8OAYlm3OAjZ4caBwAAGCDfcCeZmEV-6EDFIucDLhcwA5DkcB9DwADWUKiIUahoEgoiPniwaqJgwhmMhcDiNiUHbDwGBwIILBQAmKAqCAXrAaB7j7p6VTLk8VyMH05YgNYbJIJeqgbEgOSWLiNrcbxXQCUJIkodYAwQH01gAIwAMyiGgAAsABMykAKxoAADAAbAAnEgmk6VhADsSBIIphm2YpmA6ZpxmiMZ+nGZgnnKZgmAABy2Hi6jWMR-BkcI1hdKIywyiia4AMQADKKTp6l8MlhnqcBRAwIwUC6C8Gp0CiJVDuw1jQrOehGnSlooNaKEajKcoKgarpvqY5hWBqRIACS0FmOi3ByZ7yjyBqsBSb5VXEPj+KogQYCE6SRESdUNbaYpIHCW0zq2oHDOum7bgKe4Hk8R4nnAZ5IBeV6LfAd5Yg+T6Hpgr4fl+P7YhMaBQn0gFIPtK6XBBUGwfBiEnChaEOBhWFENDeFDYRpEpKRW0UZO1G0YG9GMV6LFsRxzJiXxknCXiohkxJLBSVTsk5PJSmqRp2l6UZpnmVZNl2Q5TkuW5HleZgPn+YFowhSkAiY5FPwxaIcXqElKVpRlWWBjleW6OtNoalk87kK07TqJ03RmP0gzDEF4yTNMpuzPMECLCsawbHwWw7Hs2TkKa8A3F19xba6lwwNctzdWKzyikIe0+5gxzdnAACC1pEs2rYvNCwGFVtwF0pOYeCgARCgzCeFAwVVGXYfQAAkqIxf52j8tQIKtBEuFbzpr6gr+i2gZtnSWv5XALzJ+Xr41xXDft136YoiAY4TlRfIZtP9eiOVlXN6whtthVOcB3cGrN4X7glxvlfWNXk9QA3TeD0GiPRW3Hfz0m6i9-3zfD-k2tjwnrXCiV9Z60A-mWJe1ZxzYzXiiUBW8QI7yfnSPedFs6BheIfcghsKAJxONGXQQCK5EloAg1gNQyF3wbjUEEzxJoBj-rlUeLxZSFGhFQ4BDdWBYLYRg3B5ADhuGipgMuOR4CpzAFkIAA