faye-authentication 1.11.0 → 1.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23d0511cded90a176f335c918ddd7ac42559d80306818da3505dbc24341b24f8
-  data.tar.gz: 3aaaa3a5e286ee2952b0098b86c6d8d10aeb84c448f606044d406baeec6d074f
+  metadata.gz: 9e267e9258a90f6c713835fa6f1f17409d717258f2e6b0d8c7aaed6b085c857b
+  data.tar.gz: 0c358ecf0dc7f03e21d61af1b172d7a3d9335f4fc9dc4a39d6499d8db8d1be71
 SHA512:
-  metadata.gz: 1e5e77a5ee7df63d0a0c62e813431baa690d16b5532a2f10494e72ca998b4393fa50f5f21adfff356c608892974a1c8b7e0fa0133d502577ef8919922df65511
-  data.tar.gz: d2bcf4df68befe130af88a1e3a94a566374257385e8b737f16c76f22a70f8a042b15d2a205302bbad658e271ad97ce16a3021b25453263ae5f672612038a0375
+  metadata.gz: d4a7e752c492860afb8fb7cda7e52a1a33be80a4db3f004526b35e5a4c0cb62b596248ec288c74d346b559048a425d2e62f65ca6edaca56344208c22ac0e57c5
+  data.tar.gz: 3fe4f26c9ad2a1a83e8b8902ed60e55f3edbd46460b43a805390f28ecb61a684d2680ed729146448ce87260153b73d341604e6a33bde580fff63e9b6ad9a9eb3

data/.gitignore

@@ -2,6 +2,7 @@
 *.rbc
 .bundle
 .config
+.ruby-version
 .yardoc
 Gemfile.lock
 InstalledFiles

data/.travis.yml

@@ -1,4 +1,5 @@
 language: ruby
 rvm:
-  - 2.1.2
-  - 1.9.3
+  - 2.6.5
+  - 2.5.7
+  - 2.4.9

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## 1.12
+  - No longer retry and fetch a new signature after errors unrelated to `Faye::Authentication` (#15)
+  - Internal:
+    - Fix rspec and jasmine specs (#14)
+    - Replace `phantomjs` with `chromeheadless` for jasmine specs (#14)
+    - Updated Travis settings with last ruby versions (#14)
+
 ## 1.11.0
   - Optional authentication for `Faye::Authentication::HTTPClient` (#12)
 

data/README.md

@@ -1,4 +1,4 @@
-# Faye::Authentication [![Build Status](https://travis-ci.org/dimelo/faye-authentication.svg?branch=master)](https://travis-ci.org/dimelo/faye-authentication) [![Code Climate](https://codeclimate.com/github/dimelo/faye-authentication.png)](https://codeclimate.com/github/dimelo/faye-authentication)
+# Faye::Authentication [![Build Status](https://travis-ci.org/jarthod/faye-authentication.svg?branch=master)](https://travis-ci.org/dimelo/faye-authentication)
 
 Authentification implementation for faye
 

data/VERSION

@@ -1 +1 @@
-1.11.0
+1.12

data/app/assets/javascripts/faye-authentication.js

@@ -8,6 +8,7 @@ function FayeAuthentication(client, endpoint, options) {
   this._waiting_signatures = [];
   this._timer = null;
   this.logger = Faye.logger;
+  this.ERROR_LIST = ['Expired signature', 'Required argument not signed', 'Invalid signature']
 }
 
 FayeAuthentication.prototype.endpoint = function() {
@@ -34,14 +35,14 @@ FayeAuthentication.prototype.resolveWaitingSignatures = function() {
         return (e.channel == params.channel && e.clientId == params.clientId);
       })[0];
       if (typeof signature === 'undefined') {
-        self.logger.error('No signature found in ajax reply for channel ' + params.channel + ' and clientId ' + params.clientId);
+        self.logger.error('[Faye] No signature found in ajax reply for channel ' + params.channel + ' and clientId ' + params.clientId);
       } else if (signature && !signature.signature) {
-        self.logger.error('Error when fetching signature for channel ' + params.channel + ' and clientId ' + params.clientId + ', error was : "' + signature.error + '"');
+        self.logger.error('[Faye] Error when fetching signature for channel ' + params.channel + ' and clientId ' + params.clientId + ', error was : "' + signature.error + '"');
       }
       FayeAuthentication.Promise.resolve(self._signatures[params.clientId][params.channel], signature ? signature.signature : null);
     });
   }, 'json').fail(function(xhr, textStatus, e) {
-    self.logger.error('Failure when trying to fetch JWT signature for data "' + JSON.stringify(messages) + '", error was : ' + textStatus);
+    self.logger.error('[Faye] Failure when trying to fetch JWT signature for data "' + JSON.stringify(messages) + '", error was : ' + textStatus);
     $.each(messages, function(key, params) {
       FayeAuthentication.Promise.resolve(self._signatures[params.clientId][params.channel], null);
     });
@@ -96,7 +97,7 @@ FayeAuthentication.prototype.authentication_required = function(message) {
       try {
         return (!this._options.whitelist(subscription_or_channel));
       } catch (e) {
-        this.logger.error("Error caught when evaluating whitelist function : " + e.message);
+        this.logger.error("[Faye] Error caught when evaluating whitelist function : " + e.message);
       }
     }
     return (true);
@@ -106,7 +107,7 @@ FayeAuthentication.prototype.authentication_required = function(message) {
 
 FayeAuthentication.prototype.incoming = function(message, callback) {
   var outbox_message = this._outbox[message.id];
-  if (outbox_message && message.error) {
+  if (outbox_message && message.error && this.ERROR_LIST.indexOf(message.error) != -1) {
     var channel = outbox_message.message.subscription || outbox_message.message.channel;
     this._signatures[outbox_message.clientId][channel] = null;
     outbox_message.message.retried = true;

data/faye-authentication.gemspec

@@ -6,11 +6,11 @@ require 'faye/authentication/version'
 Gem::Specification.new do |spec|
   spec.name          = "faye-authentication"
   spec.version       = Faye::Authentication::VERSION
-  spec.authors       = ["Adrien Siami"]
-  spec.email         = ["adrien.siami@dimelo.com"]
+  spec.authors       = ["Adrien Siami", "Adrien Rey-Jarthon", "Cyril Le Roy", "Fabien Chaynes"]
+  spec.email         = ["adrien.siami@gmail.com", "jobs@adrienjarthon.com", "cyril.leroy44@gmail.com", "fabien.chaynes@ringcentral.com"]
   spec.summary       =
   spec.description   = "A faye extension to add authentication mechanisms"
-  spec.homepage      = "https://github.com/dimelo/faye-authentication"
+  spec.homepage      = "https://github.com/jarthod/faye-authentication"
   spec.license       = "MIT"
 
   spec.files         = `git ls-files -z`.split("\x0")
@@ -21,12 +21,13 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'jwt', '>= 1.2'
   spec.add_runtime_dependency 'faye', '>= 1.0'
 
-  spec.add_development_dependency "bundler", "~> 1.5"
-  spec.add_development_dependency "rake", '~> 10.3'
-  spec.add_development_dependency 'rspec', '~> 3.0'
-  spec.add_development_dependency 'rspec-eventmachine', '~> 0.2'
-  spec.add_development_dependency 'jasmine', '~> 2.0'
-  spec.add_development_dependency 'rack', '~> 1.5'
-  spec.add_development_dependency 'thin', '~> 1.6'
-  spec.add_development_dependency 'webmock', '~> 1.18'
+  spec.add_development_dependency 'bundler'
+  spec.add_development_dependency 'rake'
+  spec.add_development_dependency 'rspec'
+  spec.add_development_dependency 'rspec-eventmachine'
+  spec.add_development_dependency 'jasmine'
+  spec.add_development_dependency 'chrome_remote'
+  spec.add_development_dependency 'rack'
+  spec.add_development_dependency 'thin'
+  spec.add_development_dependency 'webmock'
 end

data/spec/javascripts/faye-authentication_spec.js

@@ -27,8 +27,8 @@ describe('faye-authentication', function() {
   describe('authentication_required', function() {
 
     beforeEach(function() {
+      Faye.logger = {error: function() {}};
       this.auth = new FayeAuthentication(new Faye.Client('http://example.com'));
-      Faye.logger = null;
     });
 
     function sharedExamplesForSubscribeAndPublish() {
@@ -45,7 +45,6 @@ describe('faye-authentication', function() {
 
       it('logs error if the function throws', function() {
         this.auth._options.whitelist = function(message) { throw new Error("boom"); }
-        Faye.logger = {error: function() {}};
         spyOn(Faye.logger, 'error');
         this.auth.authentication_required(this.message);
         expect(Faye.logger.error).toHaveBeenCalledWith('[Faye] Error caught when evaluating whitelist function : boom');

data/spec/javascripts/faye-extension_spec.js

@@ -1,5 +1,6 @@
 describe('Faye extension', function() {
   beforeEach(function() {
+    Faye.logger = {error: function() {}};
     this.client = new Faye.Client('http://localhost:9296/faye');
     jasmine.Ajax.install();
   });
@@ -10,13 +11,15 @@ describe('Faye extension', function() {
 
   describe('Without extension', function() {
     it('fails to subscribe', function(done) {
-      this.client.subscribe('/foobar').then(undefined, function() {
+      this.client.subscribe('/foobar').then(undefined, function(e) {
+        expect(e.message).toBe('Invalid signature')
         done();
       });
     });
 
     it('fails to publish', function(done) {
-      this.client.publish('/foobar', {text: 'whatever'}).then(undefined, function() {
+      this.client.publish('/foobar', {text: 'whatever'}).then(undefined, function(e) {
+        expect(e.message).toBe('Invalid signature')
         done();
       });
     });
@@ -71,7 +74,6 @@ describe('Faye extension', function() {
         this.dispatcher = {connectionType: "fake", clientId: '1234', sendMessage: function() {}, selectTransport: function() { }};
         spyOn(this.dispatcher, 'sendMessage');
         spyOn(this.dispatcher, 'selectTransport');
-        Faye.extend(this.dispatcher, Faye.Publisher)
       });
 
       it('should add the signature to subscribe message', function(done) {
@@ -318,5 +320,44 @@ describe('Faye extension', function() {
 
     });
 
+    it('does not retry if the error is unrelated to authentication', function(done) {
+      other_client = new Faye.Client('http://localhost:9296/faye');
+      auth_extension = new FayeAuthentication(other_client, null, { retry_delay: 100 });
+      tweak_extension = {
+        incoming: function(message, callback) {
+          console.log('message',message)
+          if (message.error) {
+            message.error = 'Not an Authentication error';
+          }
+          callback(message);
+        }
+      }
+      other_client.addExtension(tweak_extension);
+      other_client.addExtension(this.extension);
+
+      jasmine.Ajax.stubRequest('/faye/auth').andReturn({
+        'responseText': '{"signature": "bad"}'
+      });
+
+      var finished = false;
+      other_client.subscribe('/toto').then(undefined, function() {
+        finished = true;
+      });
+
+      setTimeout(function() {
+
+        // Initial 200ms batching delay
+
+        setTimeout(function() {
+          expect(finished).toBe(true);
+        }, 200 + 80); // 2nd Batching delay + 80 ms
+
+        setTimeout(function() {
+          expect(finished).toBe(true);
+          done();
+        }, 200 + 200); // 2nd Batching delay + 200 ms
+      }, 200);
+    });
+
   });
 });

data/spec/javascripts/support/jasmine_helper.rb

@@ -1,13 +1,10 @@
 #Use this file to set/override Jasmine configuration options
 #You can remove it if you don't need it.
 #This file is loaded *after* jasmine.yml is interpreted.
-#
-#Example: using a different boot file.
-#Jasmine.configure do |config|
-#   config.boot_dir = '/absolute/path/to/boot_dir'
-#   config.boot_files = lambda { ['/absolute/path/to/boot_dir/file.js'] }
-#end
-#
+
+Jasmine.configure do |config|
+  config.runner_browser = :chromeheadless
+end
 
 require 'faye'
 require 'faye/authentication'

metadata

@@ -1,14 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: faye-authentication
 version: !ruby/object:Gem::Version
-  version: 1.11.0
+  version: '1.12'
 platform: ruby
 authors:
 - Adrien Siami
+- Adrien Rey-Jarthon
+- Cyril Le Roy
+- Fabien Chaynes
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-09-16 00:00:00.000000000 Z
+date: 2020-02-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jwt
@@ -42,117 +45,134 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.3'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.3'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-eventmachine
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: jasmine
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: chrome_remote
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: thin
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.18'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.18'
+        version: '0'
 description: A faye extension to add authentication mechanisms
 email:
-- adrien.siami@dimelo.com
+- adrien.siami@gmail.com
+- jobs@adrienjarthon.com
+- cyril.leroy44@gmail.com
+- fabien.chaynes@ringcentral.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -160,6 +180,7 @@ files:
 - ".drone.yml"
 - ".gitignore"
 - ".rspec"
+- ".ruby-version"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
@@ -190,7 +211,7 @@ files:
 - spec/utils/javascripts/jwt.js
 - spec/utils/javascripts/mock-ajax.js
 - spec/utils/javascripts/query-string.js
-homepage: https://github.com/dimelo/faye-authentication
+homepage: https://github.com/jarthod/faye-authentication
 licenses:
 - MIT
 metadata: {}