question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

`sentry_version` clarification

See original GitHub issue

According to the release log from 1.0.4 of raven-python, sentry_version is the expected sentry server version. [1]

However, in the unit tests and the latest official sentry Dockerfile as of today, that SENTRY_VERSION has been inconsistent specified: Dockerfile has it defined as ‘8.10’ [2] unit tests in Sentry server has it fixated at ‘2’ [3] and in raven-python fixated at ‘6’. [4]

So those brought up my question: does sentry_version actually matter in the header that raven client sent in the header? If not, can we ignore it from the header?

[1] https://github.com/getsentry/raven-python/blob/master/CHANGES#L579 [2] https://github.com/getsentry/docker-sentry/blob/master/8.10/Dockerfile#L56 [3] https://github.com/getsentry/raven-python/blob/master/tests/base/tests.py#L195 [4] https://github.com/getsentry/sentry/blob/master/tests/sentry/web/api/tests.py#L83

Issue Analytics

  • State:closed
  • Created 7 years ago
  • Comments:6 (3 by maintainers)

github_iconTop GitHub Comments

1reaction
mattrobenoltcommented, Nov 18, 2016

Correct. raven-python sends version 6: https://github.com/getsentry/raven-python/blob/master/raven/base.py#L142

Though the latest version is actually 7, we just don’t require version 7 I guess for raven-python.

0reactions
yecommented, Nov 18, 2016

Ok, so the latest sentry_version that raven-python sends in the header should be ‘6’?

https://github.com/getsentry/raven-python/blob/master/CHANGES#L320

Read more comments on GitHub >

github_iconTop Results From Across the Web

Sentry Suspect Commits Clarification - Integrations
Hello ! We have just started using sentry suspect commits and we love the fature so far. But we're having trouble understanding how...
Read more >
Sentry clarification | Dominion (Second Edition)
I'm a tiny bit not sure about text on Sentry card so this might sound like a stupid question but: Does 'any number'...
Read more >
sentry/CHANGES at master · getsentry/sentry - GitHub
chore(deps): bump @sentry/release-parser from 1.3.0 to 1.3.1 (#35657) by @dependabot ... Clarify ``Forcing documentation sync`` error messaging.
Read more >
Sentry Drones Clarification for Current Drones Control ...
In this episode of EVE Echoes, im testing SENTRY DRONE application to combat field.And for clarification - for now, sentry drones just can ......
Read more >
How does Sentry's semantic versioning work? - Stack Overflow
Also in Passing Additional Data the version is shown as "2.3.12" which is the same format as what we're providing now. sentry ·...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found