add status log (#152)

* add status log

* don't benchmark on pull_request

* feat: semantic release

.github/semantic.yml

@@ -0,0 +1,2 @@
+# Always validate the PR title AND all the commits
+titleAndCommits: true

.github/workflows/benchmark.yml

@@ -1,6 +1,5 @@
 name: Benchmark
 on:
-  pull_request:
   push:
     branches:
       - master

.github/workflows/release.yml

@@ -1,45 +1,23 @@
-name: Auto Release
-
+name: Release
 on:
   push:
-    # Sequence of patterns matched against refs/tags
-    tags:
-      - "v*" # Push events to matching v*, i.e. v1.0, v20.15.10
-
+    branches:
+      - master
 jobs:
   release:
-    name: Auto Release by Tags
+    name: Release
     runs-on: ubuntu-18.04
-
     steps:
-      - name: Checkout Repository
-        uses: actions/checkout@master
-
-      - name: Install Rust toolchain
-        uses: actions-rs/toolchain@v1
+      - name: Checkout
+        uses: actions/checkout@v1
+      - name: Setup Node.js
+        uses: actions/setup-node@v1
         with:
-          profile: minimal
-          toolchain: stable
-          default: true
-
-      - name: Cargo Login
-        uses: actions-rs/cargo@v1
-        with:
-          command: login
-          args: -- ${{ secrets.CARGO_TOKEN }}
-
-      - name: Cargo Publish
-        uses: actions-rs/cargo@v1
-        with:
-          command: publish
-
-      - name: GitHub Release
-        id: create_release
-        uses: actions/create-release@v1
+          node-version: 12
+      - name: Install dependencies
+        run: npm ci
+      - name: Release
         env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # This token is provided by Actions, you do not need to create your own token
-        with:
-          tag_name: ${{ github.ref }}
-          release_name: Release ${{ github.ref }}
-          draft: false
-          prerelease: false
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          CARGO_TOKEN: ${{ secrets.CARGO_TOKEN }}
+        run: cargo login $CARGO_TOKEN && npx semantic-release

.releaserc.json

@@ -0,0 +1,16 @@
+{
+    "debug": true,
+    "release": {
+        "branches": [
+            "master"
+        ]
+    },
+    "plugins": [
+        "@semantic-release/commit-analyzer",
+        "@semantic-release/release-notes-generator",
+        "@semantic-release/github",
+        ["@semantic-release/exec", {
+            "publishCmd" : "cargo publish"
+        }]
+    ]
+}

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "casbin"
-version = "0.8.4"
+version = "0.8.5"
 authors = ["Joey <joey.xf@gmail.com>", "Cheng JIANG <jiang.cheng@vip.163.com>"]
 edition = "2018"
 license = "Apache-2.0"

README.md

@@ -27,7 +27,7 @@ Add this package to `Cargo.toml` of your project. (Check https://crates.io/crate
 
 ```toml
 [dependencies]
-casbin = { version = "0.8.4", default-features = false, features = ["runtime-async-std", "logging"] }
+casbin = { version = "0.8.5", default-features = false, features = ["runtime-async-std", "logging"] }
 async-std = { version = "1.5.0", features = ["attributes"] }
 env_logger = "0.7.1"
 ```

src/cached_enforcer.rs

@@ -194,7 +194,7 @@ impl CoreApi for CachedEnforcer {
                         .into_iter()
                         .filter_map(|y| all_rules.get_index(y))
                         .collect();
-                    self.enforcer.get_logger().print_explain_log(rules);
+                    self.enforcer.get_logger().print_expl_log(rules);
                 }
             }
         }

src/effector.rs

@@ -10,7 +10,7 @@ pub trait Effector: Send + Sync {
 }
 
 pub trait EffectorStream: Send + Sync {
-    fn current(&self) -> bool;
+    fn next(&self) -> bool;
     fn expl(&self) -> Option<Vec<usize>>;
     fn push_effect(&mut self, eft: EffectKind) -> bool;
 }
@@ -53,7 +53,7 @@ impl Effector for DefaultEffector {
 
 impl EffectorStream for DefaultEffectStream {
     #[inline]
-    fn current(&self) -> bool {
+    fn next(&self) -> bool {
         assert!(self.done);
         self.res
     }

src/enforcer.rs

@@ -178,7 +178,7 @@ impl Enforcer {
             };
             eft_stream.push_effect(eft);
         }
-        Ok((eft_stream.current(), eft_stream.expl()))
+        Ok((eft_stream.next(), eft_stream.expl()))
     }
 }
 
@@ -417,7 +417,7 @@ impl CoreApi for Enforcer {
                         .into_iter()
                         .filter_map(|y| all_rules.get_index(y))
                         .collect();
-                    self.logger.print_explain_log(rules);
+                    self.logger.print_expl_log(rules);
                 }
             }
         }
@@ -496,6 +496,11 @@ impl CoreApi for Enforcer {
     #[inline]
     fn enable_enforce(&mut self, enabled: bool) {
         self.enabled = enabled;
+
+        #[cfg(feature = "logging")]
+        {
+            self.logger.print_status_log(enabled);
+        }
     }
 
     #[cfg(feature = "logging")]

src/logger/default_logger.rs

@@ -1,6 +1,6 @@
 use crate::{emitter::EventData, logger::Logger};
 
-use log::{error, info};
+use log::{error, info, warn};
 
 #[derive(Default)]
 pub struct DefaultLogger {
@@ -52,11 +52,23 @@ impl Logger for DefaultLogger {
         info!("{}", d);
     }
 
-    fn print_explain_log(&self, rules: Vec<&Vec<String>>) {
+    fn print_expl_log(&self, rules: Vec<&Vec<String>>) {
         if !self.is_enabled() {
             return;
         }
 
         info!("Explain: {:?}", rules);
     }
+
+    fn print_status_log(&self, enabled: bool) {
+        if !self.is_enabled() {
+            return;
+        }
+
+        if enabled {
+            info!("Casbin has been enabled!");
+        } else {
+            warn!("Casbin has been disabled!");
+        }
+    }
 }

src/logger/mod.rs

@@ -8,6 +8,7 @@ pub trait Logger: Send + Sync {
     fn enable_log(&mut self, enabled: bool);
     fn is_enabled(&self) -> bool;
     fn print_enforce_log(&self, rvals: Vec<String>, authorized: bool, cached: bool);
-    fn print_mgmt_log(&self, event_data: &EventData);
-    fn print_explain_log(&self, rules: Vec<&Vec<String>>);
+    fn print_mgmt_log(&self, d: &EventData);
+    fn print_expl_log(&self, rules: Vec<&Vec<String>>);
+    fn print_status_log(&self, enabled: bool);
 }