Menu ▾ ▴

#291 Contact server on startup

Crux v2.2
open
Kaipo
High priority (56)
2016-07-27
2015-07-01
Kaipo
No

Crux should ping a server on startup in a background thread so that we can track the number of users.

Related

Issues: #291

Discussion

  • William S Noble

    William S Noble - 2016-06-23
    • labels: --> High priority
     

  • William S Noble

    William S Noble - 2016-06-23

    If this is easy to do, it would be great to get it implemented prior to the next release.

     

  • Kaipo

    Kaipo - 2016-06-23

    We will need a URL that Crux can access which records the access. Also apart from version number and some kind of unique identifier (IP address could be enough?), is there any information we want to collect? Like maybe operating system or the command being run.

     

    • William S Noble

      William S Noble - 2016-06-23

      What info does Skyline collect? It seems like OS and command are both good
      things to collect.

      Also, we probably don't want to trigger this for all commands. Maybe just
      the ones listed as primary commands in the docs.

      Eventually, we'll want to provide a cmake option to disable this feature,
      since some people may not like it. Do you know if Skyline allows people to
      turn off the ping?

      It will also be important that the ping doesn't slow things down too much.
      Skyline just does this during startup, but we'll need to do it every time a
      command is run.

      Bill

      On Thu, Jun 23, 2016 at 11:40 AM, Kaipo kaipot@users.sf.net wrote:

      We will need a URL that Crux can access which records the access. Also
      apart from version number and some kind of unique identifier (IP address
      could be enough?), is there any information we want to collect? Like maybe
      operating system or the command being run.

      Status: open
      Milestone: Crux v2.2
      Labels: High priority
      Created: Wed Jul 01, 2015 10:06 PM UTC by Kaipo
      Last Updated: Thu Jun 23, 2016 02:46 PM UTC
      Owner: Kaipo

      Crux should ping a server on startup in a background thread so that we can
      track the number of users.

      Sent from sourceforge.net because you indicated interest in
      https://sourceforge.net/p/cruxtoolkit/issues/291/

      To unsubscribe from further messages, please visit
      https://sourceforge.net/auth/subscriptions/

       

      Related

      Issues: #291

  • Kaipo

    Kaipo - 2016-06-23

    I think this is doable - I'm not familiar with the current server setup though, maybe Charles can help with the backend part?

     

    • William S Noble

      William S Noble - 2016-06-23

      Yes, that sounds reasonable. He will be back in the office next week.
      Bill

      On Thu, Jun 23, 2016 at 3:28 PM, Kaipo kaipot@users.sf.net wrote:

      I think this is doable - I'm not familiar with the current server setup
      though, maybe Charles can help with the backend part?

      Status: open
      Milestone: Crux v2.2
      Labels: High priority
      Created: Wed Jul 01, 2015 10:06 PM UTC by Kaipo
      Last Updated: Thu Jun 23, 2016 06:56 PM UTC
      Owner: Kaipo

      Crux should ping a server on startup in a background thread so that we can
      track the number of users.

      Sent from sourceforge.net because you indicated interest in
      https://sourceforge.net/p/cruxtoolkit/issues/291/

      To unsubscribe from further messages, please visit
      https://sourceforge.net/auth/subscriptions/

       

      Related

      Issues: #291

  • Kaipo

    Kaipo - 2016-07-19

    I am attaching a patch that adds an httpGet function to Crux. Then we can contact server on a background thread with something like

    #include <boost/thread.hpp>

void server_ping() {
  httpGet("http://cruxtoolkit.sourceforge.net/examplepath/", false);
}

boost::thread ping_thread(server_ping);
     
    http.diff

    • William S Noble

      William S Noble - 2016-07-19

      So this patch doesn't actually include any calls to the function, is that
      right? It's hard to know, based on this, whether it will work as
      advertised. Is the plan to incorporate this background thread into the main
      Crux routine?

      Also, it seems like a lot of the files that are being added
      (e.g., ext/patches/boost/asio/yield.hpp) have MS-DOS EOLs.

      Bill

      On Tue, Jul 19, 2016 at 12:04 PM, Kaipo kaipot@users.sf.net wrote:

      I am attaching a patch that adds an httpGet function to Crux. Then we can
      contact server on a background thread with something like

      include <boost thread.hpp=""></boost>

      void server_ping() {
      httpGet("http://cruxtoolkit.sourceforge.net/examplepath/", false);}
      boost::thread ping_thread(server_ping);

      Attachments:

      Status: open
      Milestone: Crux v2.2
      Labels: High priority
      Created: Wed Jul 01, 2015 10:06 PM UTC by Kaipo
      Last Updated: Thu Jun 23, 2016 10:28 PM UTC
      Owner: Kaipo

      Crux should ping a server on startup in a background thread so that we can
      track the number of users.

      Sent from sourceforge.net because you indicated interest in
      https://sourceforge.net/p/cruxtoolkit/issues/291/

      To unsubscribe from further messages, please visit
      https://sourceforge.net/auth/subscriptions/

       

      Related

      Issues: #291

  • Kaipo

    Kaipo - 2016-07-19

    Right it doesn't call the function (since we don't have a server side script to access yet). I did a bit of testing by setting up an HTTP server and watching the access logs when running the function. My thought was that the background thread would go into Crux's main function.
    The added files I took directly from the asio library in the Boost distribution, since the Boost distribution included with ProteoWizard is incomplete.

     

  • Kaipo

    Kaipo - 2016-07-25

    Here is an updated patch that actually posts data to analytics.

     
    http2.diff

    • William S Noble

      William S Noble - 2016-07-25

      OK, this looks good to me. I don't have any real way to test whether the
      tracking itself is working, but I verified that the tests pass, and it
      doesn't seem to slow things down too much.

      My one request w.r.t. the code is that you add either one big or multiple
      smaller block comments into crux-utils.cpp describing how this all works. I
      also think you should add half-line comments in crux-main.cpp in the call
      to paramBuilder, explaining what each parameter is.

      On Mon, Jul 25, 2016 at 12:27 PM, Kaipo kaipot@users.sf.net wrote:

      Here is an updated patch that actually posts data to analytics.

      Attachments:

      Status: open
      Milestone: Crux v2.2
      Labels: High priority
      Created: Wed Jul 01, 2015 10:06 PM UTC by Kaipo
      Last Updated: Tue Jul 19, 2016 09:49 PM UTC
      Owner: Kaipo

      Crux should ping a server on startup in a background thread so that we can
      track the number of users.

      Sent from sourceforge.net because you indicated interest in
      https://sourceforge.net/p/cruxtoolkit/issues/291/

      To unsubscribe from further messages, please visit
      https://sourceforge.net/auth/subscriptions/

       

      Related

      Issues: #291

  • Kaipo

    Kaipo - 2016-07-25

    Updated patch with "--no-analytics <t|f>" parameter added. The "create-docs" calls that generate documentation that are part of the build use it, and the cucumber tests. I had to move the analytics code into CruxApplication::initialize, since we need to wait until after parameters have been parsed. I also added in some comments as you suggested.</t|f>

    Also, Bill you can test it by going into the analytics "Reporting" section, then Real-Time > Events, running Crux, and seeing the event appear (or alternatively generating a report under Behavior > Events.

     
    http3.diff

    • William S Noble

      William S Noble - 2016-07-26

      How do I access the Crux Google Analytics page? Do you need to give me
      access? (thabangh@gmail.com)

      Currently, I can only see the Noble lab page.

      On Mon, Jul 25, 2016 at 4:54 PM, Kaipo kaipot@users.sf.net wrote:

      Updated patch with "--no-analytics <t|f>" parameter added. The
      "create-docs" calls that generate documentation that are part of the build
      use it, and the cucumber tests. I had to move the analytics code into
      CruxApplication::initialize, since we need to wait until after parameters
      have been parsed. I also added in some comments as you suggested.</t|f>

      Also, Bill you can test it by going into the analytics "Reporting"
      section, then Real-Time > Events, running Crux, and seeing the event appear
      (or alternatively generating a report under Behavior > Events.

      Attachments:

      Status: open
      Milestone: Crux v2.2
      Labels: High priority
      Created: Wed Jul 01, 2015 10:06 PM UTC by Kaipo
      Last Updated: Mon Jul 25, 2016 07:27 PM UTC
      Owner: Kaipo

      Crux should ping a server on startup in a background thread so that we can
      track the number of users.

      Sent from sourceforge.net because you indicated interest in
      https://sourceforge.net/p/cruxtoolkit/issues/291/

      To unsubscribe from further messages, please visit
      https://sourceforge.net/auth/subscriptions/

       

      Related

      Issues: #291

      alternate

  • Kaipo

    Kaipo - 2016-07-27

    This is (mostly) committed in r17095, only a proper cid is not being posted.

     

Log in to post a comment.

MongoDB Logo MongoDB
Gen AI apps are built with MongoDB Atlas
Atlas offers built-in vector search and global availability across 125+ regions. Start building AI apps faster, all in one place.
Try Free →