add: aliases for camelCase support + container ports definition

Summary
camelCase support
The Python sidecar interface defines model properties using snake_case notation (see spec.py), however the interLink API server invokes sidecar API with objects defined in camelCase notation.
Using Pydantic alias feature, we enable Pydantic to create models parsing data using camelCase notation.
Container ports
We enriched the Container model with parameter ports defined as a list of ContainerPort models.

closing this, we are moving this library out of this repo


The port was overwritten when the server was created

Describe the bug
There is already a project for IPv4 port 3000,and when I started the project, created a project with IPv6 port 3000 as well,Previous projects will be overwritten.
Reproduction
There is an IPv4 project in the picture below, then 'vite' will still open a project with IPv6 port 3000.

System Info

vite version: 2.0.0-beta.8
Operating System: MacOS
Node version: v14.4.0

This is expected since that's Node's default behavior (IPv6 only). You'd have the same problem calling just vanilla server.listen in Node.


using serde yaml to parse config file with type check

Hi @meain , recently I was working on the symlink and theme features, finding that we were using raw rust_yaml to parse the config file.
I have tried to refactor it using serde_yaml so that we can parse the yaml file into a Rust struct, with args check, type check, and some other advanced. also we can isolate the config file logic inside the config_file modules.
for example, the config struct:
pub struct Config {
    pub classic: Option<bool>,
    pub blocks: Option<Sequence>,
    pub color: Option<Color>,
    pub date: Option<String>, // enum?
    pub dereference: Option<bool>,
    pub display: Option<String>, // enum?
    pub icons: Option<Icons>,
    pub ignore_globs: Option<Sequence>,
    pub indicators: Option<bool>,
    pub layout: Option<String>, // enum?
    pub recursion: Option<Recursion>,
    pub size: Option<String>, // enum?
    pub sorting: Option<Sorting>,
    pub no_symlink: Option<bool>,
    pub total_size: Option<bool>,
    pub styling: Option<Styling>,
}

Yeah, I had thought of doing something similar when the initial config file support was proposed.

From what I understood, you are looking up if keys exist, but I was thinking that it might be better to have a struct defined and serialize into that. But since this is not a rigid spec(as in items are optional) I am not sure if this is the right way to go.
https://github.com/Peltoche/lsd/pull/317#issuecomment-643787138

... but did not continue that discussion then. That said, I am not fully sure as to how it will work out with clap as we would ideally have to merge what we get from clap into the same struct before passing to display. We could always do what we are doing now with checking clap arguments and updating the config(but this would need us to rework flags and make config the central piece which gets updated by both config file and cli flags). Also, as @kmoschcau mentioned we will still have to write validation and serialization logic, so not a whole lot of improvement there.
I guess this does improves on the code structure, but will end up rewriting most parts of the config and flag I guess (Not sure though). @zwpaper , what are the improvements that you have in mind which this would help with?

hi @meain , as you said, it may be more universal serializing into a struct, reading yaml literally and separately in different modules is not that safe and ideally, especially in a safe language Rust😃
let's continue the discussion in this PR.

how it will work out with clap as we would ideally have to merge what we get from clap into the same struct before passing to display.

as you said, we have merged the cli and config into Flags currently, we could simply reuse the Flags, which is currently the central piece of cli and config, this should not be a problem.

we will still have to write validation and serialization logic, so not a whole lot of improvement there

adding validation is about the result, not meaning that we can leave the process in a bad way.

I guess this does improves on the code structure, but will end up rewriting most parts of the config and flag I guess (Not sure though). @zwpaper , what are the improvements that you have in mind which this would help with?

Yes, we have to rewrite much of the config logic, but this is a one time job and long term profit, after one this rewrite, we could gain more readability and maintenance.
As the improvements, let me try to express my humble opinions and we could discuss it together.

we could centralize the configuration into a struct, the struct always represents the configuration file.
isolate the yaml logic inside config_file module, the yaml knowledge is only related to config_file, it should not spread to the other modules.
type and name check, the compiler does not know anything about yaml["layout"], but it would always check config.layout, this is much safer.

all in all, I think this is a big rewrite but we can gain long-term profit with readability and maintenance.
also like the OOP said, maybe we should encapsulation the config file inside its own module, and abstraction the config logic outside

When this was first proposed, one of the main showstoppers for me was the requirement to give error output to the user when providing invalid values or other common mistakes in Yaml. Not all of those are currently in, but I don't think this would easily be possible when deserialising into a struct in one go.
For example what would happen when you expect a bool, but the user specified a string? Would the struct's member just be None in that case? You would have no clear way of knowing what went wrong then.
Yes the paths to the values in the Yaml are hard coded, but they have to be to some extent. Other than that the current deserialization handles every possible case of the value (or key) not being found, having a wrong type or an invalid value.
Other than encapsulation I don't see that much of a benefit to move all of the deserialization into one struct/namespace. But please do enlighten me if/why you think otherwise.

I haven't used structopt but I know its for unifying structs and cli args, might be helpful

hi @0jdxt thanks for the info, but we currently working on the yaml parsing, maybe the cli args could be discussed in another PR.
Hi @kmoschcau , IMHO, configuration is one of the modules of lsd, yaml is one of the implementations of the configuration. we currently added the yaml parsing, 22 of the 37 source files added the yaml related, updating this is really hard work, this is about the maintainace.
but this is not the main discussion point. let's talk about the showstoppers.
this is the error output if type mismatch when serding the config into struct:
./target/debug/lsd                                                                                [12:53:09]
configuration file format error, classic: invalid type: string "a", expected a boolean at line 1 column 10

 build.rs   Cargo.lock   Cargo.toml   CHANGELOG.md   ci   CODEOWNERS   LICENSE   README.md   src   target   tests

error key, error type, position pointed out
return None Configuration type
we can use a default Configuration when configuration file error.

Ah sorry I misunderstood the PR

BTW, I would like to drop the short ext yml support, as we support both yaml and yml automatically, but always check yml after yaml, we have to document this and users have to make sure there is not a yaml overwriting the yml one.
This may be confusing, just like the README said, it should be enough to support the yaml one, and we could tell users that always use a yaml.
what is your idea? @meain

@zwpaper I don't think having yaml and yml would cause a whole lot of confusion, but I guess supporting just one might make things simpler.

Alright, that sounds good to me. Both the serialization and only keeping the long form file extension.

Codecov Report

Merging #443 (772caf8) into master (b6bd1d3) will increase coverage by 2.00%.
The diff coverage is 88.78%.

@@            Coverage Diff             @@
##           master     #443      +/-   ##
==========================================
+ Coverage   80.71%   82.71%   +2.00%
==========================================
  Files          35       35
  Lines        3449     3258     -191
==========================================
- Hits         2784     2695      -89
+ Misses        665      563     -102

Impacted Files
Coverage Δ

src/main.rs
0.00% <0.00%> (ø)

src/flags/ignore_globs.rs
35.00% <15.38%> (-6.18%)
:arrow_down:

src/config_file.rs
76.74% <76.19%> (+58.93%)
:arrow_up:

src/flags/size.rs
92.15% <83.33%> (+3.26%)
:arrow_up:

src/flags/color.rs
92.00% <85.71%> (+0.43%)
:arrow_up:

src/flags/display.rs
92.72% <85.71%> (+1.81%)
:arrow_up:

src/flags/layout.rs
92.85% <87.50%> (+1.81%)
:arrow_up:

src/flags/icons.rs
91.72% <89.13%> (+1.80%)
:arrow_up:

src/flags/recursion.rs
93.51% <94.73%> (+4.88%)
:arrow_up:

src/flags/date.rs
90.58% <96.00%> (+2.04%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b6bd1d3...772caf8. Read the comment docs.

hi @meain this should be ready to go, could you please help to review this PR

@kmoschcau would like your review on this codebase as well if you are available since you wrote the initial config logic.

hi @kmoschcau , sorry for the rebase, I found that there is a WIP commit in my commit history and forgot that will break your review history.
I have updated the testing code according to the comment, please check it again, thanks

Yeps this looks good from my side now.

@meain I have rebased onto master, could you check this PR one more time

Hey, this looks good. @zwpaper thanks for working on this. @kmoschcau , thanks a lot for the review.
One thing I think would be good to add in the future would be to improve the error message if we expect a struct and the user provide something else. For example, if we have config like this:
color: never

the output is
lsd: Configuration file /Users/meain/.config/lsd/config.yaml format error, color: invalid type: string "never", expected struct Color at line 1 column 8.

but struct Color means nothing to the user. We should be able to implement a deserialize for those fields, something generic for all struct fields I guess which could either output something a bit more generalized like "expected a map" or just ignore that part of the message.


IDL->prose hookup should flag capitalization errors

In Typed OM, the StylePropertyMapReadOnly/size attribute dfns itself both in the IDL and in the prose, leading to a duplicate ID. I suspect the IDL->prose finder is somehow missing it because the prose dfn is for two things.

Never mind, the issue was capitalization problems - I'd spelled in "Readonly" in the prose.
I should detect these errors and flag them.


Adding .gitattributes to force unix line endings.

This is to help with issue #728

This is amazing, thank you. Would you be able to rebase these down to one commit?

For future reference, I'd have preferred a force push to this branch (neatly updating the PR in-place) rather than a brand new one.


Run a Custom Install

Custom-Run Install Profiles
I'd like to make sure that all of the features I want to be installed with a particular Xcode version are properly recognized by the Xcodes app as being part of said install, even when all of the installation files are downloaded manually from Apple. For example, in browsing the More Downloads section of the Apple Developer website, I found exactly what I was looking for, in terms of packaged components of Xcode, as well as the Xcode app itself, that I wanted to install, but I am worried that, given there is no way to give these files to the Xcodes app to be installed explicitly side-by-side to my Mac App Store Xcode install, that I am either going to accidentally replace my Mac App Store version with this downloaded one, or that Xcodes will not properly recognize all of the components I want associated with this separate Xcode install, such as the Additional Tools package and others.
Details
It would be nice to simply have a feature where you can fill out a form in the app that asks you for images for various components, and then associates them all to one particular install "profile", where you can have multiple of the same version, if, for example, you want to have a different setup for a different workload, and maybe want a different version of some additional package available while the profile is active, but can also use this to make sure you have all the components you want available, such as the latest version of the Xcode beta and its additional packages, all associated together as an install "profile".

Even just being able to present an .xcdownloadmanifest such as below to Xcodes would be super nice.
{
  "parentFile": "Xcode_15_beta.xip",
  "relatedAssets": [
    "iOS_17_beta_Simulator_Runtime.dmg",
    "watchOS_10_beta_Simulator_Runtime.dmg",
    "tvOS_17_beta_Simulator_Runtime.dmg"
  ]
}

Will now also note that I tried installing Xcode 15 beta from the files I downloaded, as detailed above, with some more optional packages, and none of it shows up in the Xcodes app... So kind of even worse than what I was expecting (aside: may open another issue for this, but install seems to be localized to Xcode-beta.app to some extent, so I think I might just keep it like this for now and tread lightly). Would be really nice to have this.


Feature/footer

Finish footer component. Ready for review.

Unified class names to CamelCase convention, added missing alt tags to images. Added mixins for responsive design.
Ready for re-testing.


FreeRadius with winbind/mschap for AD auth

Describe the bug
i'm trying to get freeradius with AD via mschap working
Steps To Reproduce
Steps to reproduce the behavior:

Install freeradius
enable the mschap module

Error: /etc/raddb/mods-enabled/mschap[11]: 'winbind' is not enabled in this build.
Error: /etc/raddb/mods-enabled/mschap[11]: Instantiation failed for module "mschap"

Expected behavior
i was expecting that support for mschap/winbind is built-in
Additional context
i tried to overwrite
nixpkgs.overlays = [
  (self: super: {
    freeradius = super.freeradius.overrideAttrs (oldAttrs: {
      nativeBuildInputs = oldAttrs.buildInputs ++ [ "winbind" ];
    });
  })
];

and the build failed. but I'm pretty new to this overriding stuff
https://wiki.freeradius.org/guide/Active-Directory-direct-via-winbind#compiling-and-installing-freeradius
is talking about --with-winbind-dir=/opt/samba4.2 i guess that needs to be added?
Notify maintainers
@sheenobu @WilliButz @AndersonTorres
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.29, NixOS, 23.05 (Stoat), 23.05.20230518.48a0fb7`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I do not maintain Freeradius.

I do not maintain Freeradius.

yes i know but i found you via git blame, you added the todo and i hoped you maybe know how to. sorry for the ping and thanks a lot for the quick answer

The blame is a bit misleading.
I merely refactored rewrote the code because I did a treewide change - namely, change finger_bsd to bsd-finger.
The TODO lines were indeed written by @ sheenobu.
Sorry for not being of so much help.


how to find out if user has logged out of FB?

i need to log the user out of my app when that person logs out of his facebook account, i don't seem to find a way to discover this?
I tried to use current_facebook_user but it is never set to nil. I tried to use current_facebook_user.fetch and it seems to throw an exception if the user logs out - but its erratic. Does not behave like that all the time for all users.
what is the correct way to discover if the user is still logged in facebook or not?

The javascript API will update the cookies. If you aren't using the javascript api, you just need to handle the exceptions.
Mike
On Oct 25, 2011, at 10:42 AM, David Ang wrote:

i need to log the user out of my app when that person logs out of his facebook account, i don't seem to find a way to discover this?
I tried to use current_facebook_user but it is never set to nil. I tried to use current_facebook_user.fetch and it seems to throw an exception if the user logs out - but its erratic. Does not behave like that all the time for all users.
what is the correct way to discover if the user is still logged in facebook or not?
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64

--
Mike Mangino
http://www.elevatedrails.com

Hi Mike
Thanks for the fast response. I am using the javascript api - i had your helper on my page: fb_connect_async_js. See code - http://d.pr/vw3c
Am I suppose to be adding something else? Should i use be using  FB.getLoginStatus  to check for login? (its always false when i tried).
My code is really minimal - using your helper fb_connect_async_js and fb_login_and_redirect. On my application code, I simply use current_facebook_user and current_facebook_user.fetch to check if the user can login to FB or not. Its really erratic some of my peers logs out of Facebook and current_facebook_user.fetch still succeeds.

Welcome to the world of Facebook. There is no really good way to test things. On the web side, a page reload is needed to detect logout, and it may actually take two pages. I typically just ignore logging out and handle the exceptions from Facebook globally.
Mike
On Oct 25, 2011, at 12:18 PM, David Ang wrote:

Hi Mike
Thanks for the fast response. I am using the javascript api - i had your helper on my page: fb_connect_async_js. See code - http://d.pr/vw3c
Am I suppose to be adding something else? Should i use be using  FB.getLoginStatus  to check for login? (its always false when i tried).
My code is really minimal - using your helper fb_connect_async_js and fb_login_and_redirect. On my application code, I simply use current_facebook_user and current_facebook_user.fetch to check if the user can login to FB or not. Its really erratic some of my peers logs out of Facebook and current_facebook_user.fetch still succeeds.
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519253

--
Mike Mangino
http://www.elevatedrails.com

Oh - so this kind of behavior is normal and is expected? So then its good to know im using your library correctly and nothing is wrong on my end.
Thanks for the advise Mike.
On Oct 26, 2011, at 12:21 AM, Mike Mangino wrote:

Welcome to the world of Facebook. There is no really good way to test things. On the web side, a page reload is needed to detect logout, and it may actually take two pages. I typically just ignore logging out and handle the exceptions from Facebook globally.
Mike
On Oct 25, 2011, at 12:18 PM, David Ang wrote:

Hi Mike
Thanks for the fast response. I am using the javascript api - i had your helper on my page: fb_connect_async_js. See code - http://d.pr/vw3c
Am I suppose to be adding something else? Should i use be using  FB.getLoginStatus  to check for login? (its always false when i tried).
My code is really minimal - using your helper fb_connect_async_js and fb_login_and_redirect. On my application code, I simply use current_facebook_user and current_facebook_user.fetch to check if the user can login to FB or not. Its really erratic some of my peers logs out of Facebook and current_facebook_user.fetch still succeeds.
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519253

--
Mike Mangino
http://www.elevatedrails.com
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519286

Yeah, Facebook kind of hacked authentication together. There is no good way to coordinate logout, so they just do it halfway. It's one of many reasons I try to do as little with Facebook as possible.
Mike
On Oct 25, 2011, at 12:37 PM, David Ang wrote:

Oh - so this kind of behavior is normal and is expected? So then its good to know im using your library correctly and nothing is wrong on my end.
Thanks for the advise Mike.
On Oct 26, 2011, at 12:21 AM, Mike Mangino wrote:

Welcome to the world of Facebook. There is no really good way to test things. On the web side, a page reload is needed to detect logout, and it may actually take two pages. I typically just ignore logging out and handle the exceptions from Facebook globally.
Mike
On Oct 25, 2011, at 12:18 PM, David Ang wrote:

Hi Mike
Thanks for the fast response. I am using the javascript api - i had your helper on my page: fb_connect_async_js. See code - http://d.pr/vw3c
Am I suppose to be adding something else? Should i use be using  FB.getLoginStatus  to check for login? (its always false when i tried).
My code is really minimal - using your helper fb_connect_async_js and fb_login_and_redirect. On my application code, I simply use current_facebook_user and current_facebook_user.fetch to check if the user can login to FB or not. Its really erratic some of my peers logs out of Facebook and current_facebook_user.fetch still succeeds.
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519253

--
Mike Mangino
http://www.elevatedrails.com
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519286

--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519507

--
Mike Mangino
http://www.elevatedrails.com

Unfortunately for us our game relies solely on Facebook.  All our players are facebook users!
On Oct 26, 2011, at 12:41 AM, Mike Mangino wrote:

Yeah, Facebook kind of hacked authentication together. There is no good way to coordinate logout, so they just do it halfway. It's one of many reasons I try to do as little with Facebook as possible.
Mike
On Oct 25, 2011, at 12:37 PM, David Ang wrote:

Oh - so this kind of behavior is normal and is expected? So then its good to know im using your library correctly and nothing is wrong on my end.
Thanks for the advise Mike.
On Oct 26, 2011, at 12:21 AM, Mike Mangino wrote:

Welcome to the world of Facebook. There is no really good way to test things. On the web side, a page reload is needed to detect logout, and it may actually take two pages. I typically just ignore logging out and handle the exceptions from Facebook globally.
Mike
On Oct 25, 2011, at 12:18 PM, David Ang wrote:

Hi Mike
Thanks for the fast response. I am using the javascript api - i had your helper on my page: fb_connect_async_js. See code - http://d.pr/vw3c
Am I suppose to be adding something else? Should i use be using  FB.getLoginStatus  to check for login? (its always false when i tried).
My code is really minimal - using your helper fb_connect_async_js and fb_login_and_redirect. On my application code, I simply use current_facebook_user and current_facebook_user.fetch to check if the user can login to FB or not. Its really erratic some of my peers logs out of Facebook and current_facebook_user.fetch still succeeds.
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519253

--
Mike Mangino
http://www.elevatedrails.com
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519286

--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519507

--
Mike Mangino
http://www.elevatedrails.com
--
Reply to this email directly or view it on GitHub:
https://github.com/mmangino/facebooker2/issues/64#issuecomment-2519562

You can ask Facebook for a method, but at this point, there's nothing we can do to make this work. It's a facebook limitation.


how can i create new clothing location and job clothing areas

how can i create new clothing location and job clothing areas

This is not a place to ask for help. Its pretty easy to do by looking at shared/config.lua and duplicating what's already there.
If you still can't figure it out, join the discord and create a support post there. Closing.


Simple docs for the golang sdk

Short README on the methods that the handlers can support - will follow up with another PR for the examples

Can we do something like https://pkg.go.dev/golang.org/x/tools/cmd/godoc ? Might be overkill, but if it's easy and cheap to set up may be worth it?

yea may be overkill for this, we can do it if requested by people in the future


bugfix Set the enable_fallback_to_original_planner behavior

Proposed changes
In the latest master code, enable_fallback_to_original_planner no longer exists, so the return value needs to be processed.
related #40440

run buildall


Providing custom message when failed to extract token

Are there any support for this ?
passport.use('jwt', new JwtStrategy({
        secretOrKey: authConfig.secret,
        jwtFromRequest: ExtractJwt.fromAuthHeader(), //If return null, finishes with empty message
        algorithms: authConfig.algorithms
    },
    function(jwt_payload, done) {
        //Never had the chance to call done with custom message
        Account.findById(jwt_payload.userId, function(err, account) {
            if (err) return done(err);
            if (!account) return done(null, false, 'invalid Token');
            return done(null, account, 'valid Token');
        });
    })
);

I think you're looking for the Custom Callback functionality of the Passport module.

Sure, I can go and check if the authorization header exists before triggering the passport middleware. But why don't we put it into the strategy extraction method itself? Something like:
jwtFromRequest: ExtractJwt.fromAuthHeader({failmessage: 'missing token'})

This is not the point of the extractor function and a violation of single responsibility pattern.  The extractor function is only responsible for extracting the JWT from the request if it exists.
There is no need to check the request for the authorization header before triggering passport. The decision of what to do if the JWT is missing takes places in the strategy's authenticate() method..  Right now it returns a generic error which you can then catch and handle in the custom callback register via  passports Custom Callback functionality.
That is where I recommend providing the custom error response.
The potential improvements I see in this area are:

Return a custom error class to make it easier to identify when the JWT was missing.
Do something like passport-local and provide a custom error message via the Strategy constructors generic options argument.

Let me know if this covers your use case.  If you have a suggestion I am open to pull requests or we can create a feature request to address the need.
Thanks for using the module and I appreciate any feedback!.

Haven't heard anything in a few days. Closing for now.

Hello. Cannot understand how to set custom error message for different errors and how can I embed JWT black list to this strategy? Thanks for answer

@GeekEdem Sorry to sound like a broken record but handling custom error messages is explained here: http://passportjs.org/docs#custom-callback
and if you want to blacklist JWTs you can put that logic into the verify callback when constructing the strategy
passport.use(new JwtStrategy(opts, function(jwt_payload, done) {
       if ( is_jwt_blacklisted(jwt_payload) ) {
            return done(err, false);
        }
        else  {
            // jWT is not blacklisted, look up a user or
           // whatever else you were planning on doing with a
          // valid JWT
        }
    });

note that is_jwt_blacklisted is just a funciton I made up. You'll have to supply that function as the strategy doesn't maintain a blackisted jwt list on its own.

@themikenicholson Here is my solution, is some one need)
function requireAuth (req, res, next){
    passport.authenticate('jwt', jwtSession, function (error, decryptToken, jwtError) {
        if(typeof (jwtError) === 'object'){
            return general.response(res, {
                field: 'Authorization',
                location: 'header',
                messages: [
                    jwtError.message
                ]
            });
        } else if (!error) {
            let token = req.header('Authorization').slice(4);
            TokenModel.findOne({token: token}).lean().exec( (err, result) => {
                if(!err && !result) {
                    req.user = decryptToken;
                    return next();
                } else if (!err && result) general.response(res, {
                    field: 'Authorization',
                    location: 'header',
                    messages: [
                        'token is in black list'
                    ]
                });
                else general.response(res, err);
            });
        }
    })(req, res, next);
}

this method I use as middleware.
general.response - this is method, that make response message and return them.
TokenModel - this is mongoose model, to check is token in blacklist collection in MongoDB.

@themikenicholson
Following the passport custom-callback
router.get('/jwt', (req, res, next) => {
    passport.authenticate('jwt', (err, account, info) => {
        if (err) return next(err); // It is null
        if (!account) return res.status(403).json(info);
        res.status(200).json(account);
    })(req, res, next);
});

If token extraction failed, the custom error is placed in the "info" parameter. Is this by design? Shouldn't it be at the "err" parameter?

You can check for Error string in info to catch and provide custom messages.
passport.authenticate('jwt', (error, user, info) => {
    sails.log.info('isAuthenticated policy: ', error, user, info.name);

    if (info.name === 'TokenExpiredError') info.status = 401;
    if (info.name === 'JsonWebTokenError') info.status = 401;
    if (info.name === 'Error') info.status = 401;
    if (error || !user) return res.negotiate(error || info);

    req.user = user;
    next();

  })(req, res);

Manteiners should add this "info" param and examples to the http://www.passportjs.org/packages/passport-jwt/ docs. I could't find the error until i read this issue. Thanks


python 3.10.5 pip install fails on not installing 'requests' before usage

When installing the fortigate-api using pip, it crashes and fails on 'requests' not being present.
    × Getting requirements to build wheel did not run successfully.
    │ exit code: 1
    ╰─> [21 lines of output]
        Traceback (most recent call last):
          File "/tmp/tmp9j5_wgaq_in_process.py", line 363, in <module>
            main()
          File "/tmp/tmp9j5_wgaq_in_process.py", line 345, in main
            json_out['return_val'] = hook(**hook_input['kwargs'])
          File "/tmp/tmp9j5_wgaq_in_process.py", line 130, in get_requires_for_build_wheel
            return hook(config_settings)
          File "/tmp/pip-build-env-6rdrkzvm/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 177, in get_requires_for_build_wheel
            return self._get_build_requires(
          File "/tmp/pip-build-env-6rdrkzvm/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 159, in _get_build_requires
            self.run_setup()
          File "/tmp/pip-build-env-6rdrkzvm/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 281, in run_setup
            super(_BuildMetaLegacyBackend,
          File "/tmp/pip-build-env-6rdrkzvm/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 174, in run_setup
            exec(code, locals())
          File "<string>", line 7, in <module>
          File "/tmp/pip-req-build-psl84rhi/fortigate_api/__init__.py", line 3, in <module>
            from fortigate_api.fortigate import Fortigate
          File "/tmp/pip-req-build-psl84rhi/fortigate_api/fortigate.py", line 12, in <module>
            import requests
        ModuleNotFoundError: No module named 'requests'
        [end of output]

Using a fresh virtualenv the following can be seen. I'm not able to understand yet, why pypi wouldn't honor the "install_requires" argument (or so it seems).
hostname ~/tmp
$ venvcreate
Requirement already satisfied: pip in ./venv/lib/python3.10/site-packages (22.0.4)
Collecting pip
  Using cached pip-22.2.1-py3-none-any.whl (2.0 MB)
Requirement already satisfied: setuptools in ./venv/lib/python3.10/site-packages (58.1.0)
Collecting setuptools
  Using cached setuptools-63.3.0-py3-none-any.whl (1.2 MB)
Installing collected packages: setuptools, pip
  Attempting uninstall: setuptools
    Found existing installation: setuptools 58.1.0
    Uninstalling setuptools-58.1.0:
      Successfully uninstalled setuptools-58.1.0
  Attempting uninstall: pip
    Found existing installation: pip 22.0.4
    Uninstalling pip-22.0.4:
      Successfully uninstalled pip-22.0.4
Successfully installed pip-22.2.1 setuptools-63.3.0
(venv) hostname ~/tmp
$ pip install fortigate-api
Collecting fortigate-api
  Using cached fortigate_api-0.2.5.tar.gz (30 kB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [10 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-dsvb5kv5/fortigate-api_01b828e11cff462d9af5d9c172bd885d/setup.py", line 7, in <module>
          import fortigate_api as package
        File "/tmp/pip-install-dsvb5kv5/fortigate-api_01b828e11cff462d9af5d9c172bd885d/fortigate_api/__init__.py", line 3, in <module>
          from fortigate_api.fortigate import Fortigate
        File "/tmp/pip-install-dsvb5kv5/fortigate-api_01b828e11cff462d9af5d9c172bd885d/fortigate_api/fortigate.py", line 12, in <module>
          import requests
      ModuleNotFoundError: No module named 'requests'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

python version: 3.10.5
prior versions don't seem to have this issue.

I can't reproduce this problem

It seems when creating this I left out an important detail. I was using poetry to install fortigate-api.
mkdir /tmp/ff
python3.10 -m venv .venv
poetry init
poetry add fortigate-api

output:
rsion ^0.2.6 for fortigate-api

Updating dependencies
Resolving dependencies... (0.6s)

Writing lock file

Package operations: 6 installs, 0 updates, 0 removals

  • Installing certifi (2022.6.15)
  • Installing charset-normalizer (2.1.1)
  • Installing idna (3.3)
  • Installing urllib3 (1.26.12)
  • Installing requests (2.28.1)
  • Installing fortigate-api (0.2.6): Failed

  CalledProcessError

  Command '['/private/tmp/ff/.venv/bin/python', '/Users/bart/Library/Application Support/pypoetry/venv/lib/python3.10/site-packages/virtualenv/seed/wheels/embed/pip-22.2.2-py3-none-any.whl/pip', 'install', '--use-pep517', '--disable-pip-version-check', '--prefix', '/private/tmp/ff/.venv', '--no-deps', '/Users/bart/Library/Caches/pypoetry/artifacts/83/37/8f/35d181b16e27e34c1900d827ecfea7867d3fe0db40ba7bc12b5921af42/fortigate_api-0.2.6.tar.gz']' returned non-zero exit status 1.

  at ~/.pyenv/versions/3.10.5/lib/python3.10/subprocess.py:524 in run
       520│             # We don't call process.wait() as .__exit__ does that for us.
       521│             raise
       522│         retcode = process.poll()
       523│         if check and retcode:
    →  524│             raise CalledProcessError(retcode, process.args,
       525│                                      output=stdout, stderr=stderr)
       526│     return CompletedProcess(process.args, retcode, stdout, stderr)
       527│
       528│

The following error occurred when trying to handle this error:

  EnvCommandError

  Command ['/private/tmp/ff/.venv/bin/python', '/Users/bart/Library/Application Support/pypoetry/venv/lib/python3.10/site-packages/virtualenv/seed/wheels/embed/pip-22.2.2-py3-none-any.whl/pip', 'install', '--use-pep517', '--disable-pip-version-check', '--prefix', '/private/tmp/ff/.venv', '--no-deps', '/Users/bart/Library/Caches/pypoetry/artifacts/83/37/8f/35d181b16e27e34c1900d827ecfea7867d3fe0db40ba7bc12b5921af42/fortigate_api-0.2.6.tar.gz'] errored with the following return code 1, and output:
  Processing /Users/bart/Library/Caches/pypoetry/artifacts/83/37/8f/35d181b16e27e34c1900d827ecfea7867d3fe0db40ba7bc12b5921af42/fortigate_api-0.2.6.tar.gz
    Installing build dependencies: started
    Installing build dependencies: finished with status 'done'
    Getting requirements to build wheel: started
    Getting requirements to build wheel: finished with status 'error'
    error: subprocess-exited-with-error

    × Getting requirements to build wheel did not run successfully.
    │ exit code: 1
    ╰─> [21 lines of output]
        Traceback (most recent call last):
          File "/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/tmpxqohejrd_in_process.py", line 363, in <module>
            main()
          File "/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/tmpxqohejrd_in_process.py", line 345, in main
            json_out['return_val'] = hook(**hook_input['kwargs'])
          File "/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/tmpxqohejrd_in_process.py", line 130, in get_requires_for_build_wheel
            return hook(config_settings)
          File "/private/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/pip-build-env-0pyfyeip/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 338, in get_requires_for_build_wheel
            return self._get_build_requires(config_settings, requirements=['wheel'])
          File "/private/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/pip-build-env-0pyfyeip/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 320, in _get_build_requires
            self.run_setup()
          File "/private/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/pip-build-env-0pyfyeip/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 482, in run_setup
            super(_BuildMetaLegacyBackend,
          File "/private/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/pip-build-env-0pyfyeip/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 335, in run_setup
            exec(code, locals())
          File "<string>", line 7, in <module>
          File "/private/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/pip-req-build-1la2_xk_/fortigate_api/__init__.py", line 3, in <module>
            from fortigate_api.fortigate import Fortigate
          File "/private/var/folders/4c/xmcy7hkn283bzd0vthnj87mw0000gn/T/pip-req-build-1la2_xk_/fortigate_api/fortigate.py", line 12, in <module>
            import requests
        ModuleNotFoundError: No module named 'requests'
        [end of output]

    note: This error originates from a subprocess, and is likely not a problem with pip.
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip.

  at ~/Library/Application Support/pypoetry/venv/lib/python3.10/site-packages/poetry/utils/env.py:1473 in _run
      1469│                 output = subprocess.check_output(
      1470│                     command, stderr=subprocess.STDOUT, env=env, **kwargs
      1471│                 )
      1472│         except CalledProcessError as e:
    → 1473│             raise EnvCommandError(e, input=input_)
      1474│
      1475│         return decode(output)
      1476│
      1477│     def execute(self, bin: str, *args: str, **kwargs: Any) -> int:

The following error occurred when trying to handle this error:

  PoetryException

  Failed to install /Users/bart/Library/Caches/pypoetry/artifacts/83/37/8f/35d181b16e27e34c1900d827ecfea7867d3fe0db40ba7bc12b5921af42/fortigate_api-0.2.6.tar.gz

  at ~/Library/Application Support/pypoetry/venv/lib/python3.10/site-packages/poetry/utils/pip.py:51 in pip_install
       47│
       48│     try:
       49│         return environment.run_pip(*args)
       50│     except EnvCommandError as e:
    →  51│         raise PoetryException(f"Failed to install {path.as_posix()}") from e
       52│

(.venv) bart@Barts-MacBook-Pro ff % python3 -V                                                                            1 ↵
Python 3.10.6

problem solved in fortigate-api 1.0.0


%%opts doesn't seem to have any effect when using a Dataset (matplotlib backend)

As an example, running the following cell in a notebook
%%opts Image style(interpolation='bilinear', cmap='viridis') plot[colorbar=True]
hv_ds = hv.Dataset(xarray_dataset, vdims=['topographic__elevation'])
hv_ds.to.image(['x', 'y'])

still gives the default cmap and no colorbar:

(tested with holoviews 1.6.2, matplotlib 1.5 and 2.0 and xarray 0.9.1)

This is a bit confusing and is something that will hopefully be resolved soon. We currently have both an Image Element and a GridImage Element, the latter exists to have support for grid based datasets like xarray. They will be unified in #994, but for now you have to specifically refer to GridImage.

Works perfectly, thanks!!
I'm fine using GridImage instead of Image.
BTW, great package, good job!

BTW, great package, good job!

Glad it's working for you, and thanks a lot. Feel free to ask any questions!


Set up GitHub Actions to compile PathVisio

Fix #202

The Travis CI configuration also built against OpenJDK 11 and 14. This uses Temurin, which does not have 14. However, neither 11 nor 17 (the next LTS in Temurin) compile.

ah perfect! thanks! just got confused... :)


Profile system performance: Incorrect default value

The default value for this user right is stated to be Administrators but the correct default value is Administrators and NT SERVICE\WdiServiceHost. I've verified this on Windows Server 2012 R2, Windows Server 2016, and Windows 10, but believe this is the default value as far back as Windows Vista and Server 2008? It presumably also still holds true for Windows Server 2019.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f767b9ed-281b-97cf-b59c-5bdd7f2c10b5
Version Independent ID: 6e9ec3c3-62eb-f71c-f74f-75a61a53a9d5
Content: Profile system performance (Windows 10)
Content Source: windows/security/threat-protection/security-policy-settings/profile-system-performance.md
Product: w10
Technology: windows
GitHub Login: @Justinha
Microsoft Alias: justinha

@officedocsbot assign @mypil

@ralish This is confirmed that this value holds for Windows Server 2019 as well.

@ralish - Thank you for submitting this feedback.
From our understanding, the issue has been resolved. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Degraded performance

Hi,
We are seeing performance issues with feed going through feedrapp. This all started Monday (Mar 27).
Fiddler shows 14 sec response time, any suggestions to improve this?
Appreciate your help.
Thanks

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Hello, I've also noticed slow load times with feedrapp. Starting March 27 the load time has slowed to 15 sec to 20 sec.
A fix would be greatly appreciated!
Thanks

Hi there,
We have been having issues with the load time since Monday as well. Do you have any suggestions on how to fix the load time? is there any fix?
Any help would be appreciated.
Thank you

Hm interesting. Can you pls tell me the feed URL?

Here the feed that I have been using: https://www.smartbrief.com/servlet/rss?b=ACC
This had been working perfect till last week.

Some context: I migrated from heroku to aws and expected some actual perf improvement but maybe I've been wrong with that assumption...

Are you planning on bringing back to Heroku?
Thanks

I will think about the options over the weekend and will also debug the issue a bit.

The feeds are not working at all now, have you made an update?

Should be fine now. The HDD ran full. :(

Alright. Can you please tell me if the issue is resolved?

It is working again, for me.

All working like a charm now. Thank you very much!

Forgot to say this: thank you.

Thank You, it is working fine now.
Appreciate your help.

It's working again and it's way faster than before. Thank you so much :)

All fixed, thanks!

:)
jeremyjburnett<EMAIL_ADDRESS>schrieb am Mo. 3. Apr. 2017 um
19:43:

All fixed, thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sdepold/jquery-rss/issues/127#issuecomment-291218036,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE1O6BWcTK1m3_pwfwZJlFRMmx9FjLwks5rsS-sgaJpZM4Mup7M
.

:)

Looks like the issue is back again, it is slow. Can you check?

I think the server is simply not capable of handling the load ... It eats up a quite significant amount of memory 🤔  Will look into it!

Aaaaalright. I just moved the service to a new server which seems to have no bigger issues with the load and is relatively cheap. Can you please tell me how everythings works performance-wise for you?

Right now, at least for me, it is pretty much instant and without issues.
Thank you and we all appreciate your work and effort.

It's happening again. US west coast.

Should be better :)

Hey everybody.
I made a rather subtle change in the feedr server which I hope has a huge impact on the stability of the server. You can find the change here: https://github.com/sdepold/feedrapp/pull/71
TLDR: I moved the caching layer far up in the request hierarchy and improved the performance of the cache hits by 100-1000 times. A cache hit was previously taking around 13ms to get served. It is now down to 0.01ms. I'd love to get some feedback about the performance from you!

Hello sdepold, i wants to use my own host server for run feedr. Do you have some guide or tutorial ?
"...If you don't want to rely on the provided server and instead run your own version, you can just download feedr, install the dependencies and run it..."
Where i can download feedr¿ and how i can isntall on the server?
Thank you.

Pls check http://www.feedrapp.info/#hosting-section

Hello sdepold, there isnt a link to download the app? i dont know how to start ... i never do this =)

I'll close this issue for now since everything seems to work fine. feel free to reopen


chore: deprecate observable symbol export

resolves #6489

Core Team:

Yes to deprecation.
Probably should rename it.

@cartant will have a look.

Ping @cartant

Core team: Ping @cartant <3


Enhance Search Functionality to Handle Both Place Names and DIGIPIN

Description
This PR enhances the search functionality in the DigiPinMap component to handle both place names and DIGIPINs using a single input field. The updates include:

Input Handling:

Modified the existing input field to handle both place names and DIGIPINs.
Added logic to determine if the input is a place name or a DIGIPIN.

Search Functionality:

If the input is a place name, the Mapbox Geocoding API is used to get the coordinates.
If the input is a DIGIPIN, the Get_LatLong function is used to get the coordinates.

Screenshots

Fix #4

I’ve updated the function to retrieve the latitude and longitude value from Digipin, incorporating the new logic for generating Digipin.


Creating Shortcuts

Do I need to use a plugin?

What do you mean by shortcuts?

App shortcuts on the homescreen

Do you mean like multiple entry points into the same app via different launcher icons?

Well, after installation of the app, I will like a shortcut to be created on the screen homepage, currently it's only created at the apps list area only.

Did you use the Tabris.js build service to build your own app?
If you build an app out of your Tabris.js JavaScript code, that app can have its own launcher icon. You can supply your own branding. See here: http://eclipsesource.com/blogs/2015/04/02/brand-your-tabris-js-app/

Apps like whatsapp create a shortcut on the main homepage (screen) on
android. No homescreen shortcut after I install my build.
On Wed, May 6, 2015 at 8:49 AM, Johannes Eickhold<EMAIL_ADDRESS>wrote:

Did you use the Tabris.js build service to build your own app?
If you build an app out of your Tabris.js JavaScript code, that app can
have its own launcher icon. You can supply your own branding. See here:
http://eclipsesource.com/blogs/2015/04/02/brand-your-tabris-js-app/
--
Reply to this email directly or view it on GitHub
https://github.com/eclipsesource/tabris-js/issues/433#issuecomment-99385683
.

If there is no space on your homescreen the shortcut is created on another screen (if there is place left there). If all homescreens are full you can open your appdrawer and drag&drop your app to the homescreen. If there is no space left, you can group apps. Or you can remove another shortcut first and then drop you Tabris.js app there. At least that is how it works on my Android phone. Does this help?

Yeah I do that....just wanted an automated way.
On Wed, May 6, 2015 at 3:58 PM, Johannes Eickhold<EMAIL_ADDRESS>wrote:

If there is no space on your homescreen the shortcut is created on another
screen (if there is place left there). If all homescreens are full you can
open your appdrawer and drag&drop your app to the homescreen. If there is
no space left, you can group apps. Or you can remove another shortcut first
and then drop you Tabris.js app there. At least that is how it works on my
Android phone. Does this help?
--
Reply to this email directly or view it on GitHub
https://github.com/eclipsesource/tabris-js/issues/433#issuecomment-99522633
.

@ruarch IMHO the functionality is out-of-scope for Tabris.js. Creating shortcuts on the homescreen is functionality provided by you mobile operating system. When you install a Tabris.js app for the first time and there is space left on one of you screens, a shortcut with the app icon is created automatically.


CdnSupportsSsl has no purpose and should be implemented

CdnSupportsSsl has no purpose and should be implemented. The key issue is that some CDN's don't support SSL and in this case the resource should fall back on the local resource, which can naturally be served under the same scheme as the site.
Note: an example scenario is AzureCDN with a custom domain, even though this is set to change 2016Q2 as per https://feedback.azure.com/forums/169397-cdn/suggestions/1332683-allow-https-for-custom-cdn-domain-names in case you care to vote!)
Requirements

If the CDN does support SSL, it is greatly encouraged to serve resources from the CDN under SSL, even if the site is not. This also ensures that, when a page such as "Contact Us" is served under SSL, these resources do not have to be reloaded.
If the CDN does not support SSL but the site is served under SSL, then we should never attempt to load resources from the CDN. The main current browsers will block the resource entirely. The logical fallback is to load the resource locally.

Specifying Resources

When the CDN supports SSL, all that is required is for the URL's passed into SetCdn to start with "https". No need to configure CdnSupportsSsl - its value will be set automatically.
When the CDN does not support SSL, the full overload of SetCdn should specify "false" for the CdnSupportsSsl parameter. Logically, the URL's passed in start with "http".
Using protocol relative links is discouraged, if the CDN supports SSL, then always configure the resource's CDN URL's as https.

Implementation

Pass the Request.IsSecureConnection from CoreShapes as a parameter to the ResourceRequiredContext,
If CdnSupportsSsl is not provided, then set it to true if CdnUrl starts with "https",
If serving under SSL and CdnSupportsSsl and CDN URL is configured then return CDN URL,
If  not serving under SSL and CDN URL is configured then return CDN URL
Otherwise return local URL

Pull request #6832


Feature/autodiscovery customroutes

Closes #108
What do you think @Triloworld

Codecov Report

Merging #109 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #109   +/-   ##
=======================================
  Coverage   36.92%   36.92%
=======================================
  Files           5        5
  Lines          65       65
  Branches        7        7
=======================================
  Hits           24       24
  Misses         38       38
  Partials        3        3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0d3f65b...d8627ef. Read the comment docs.

Yeah ! Working fine. Tested auto-discovery and custom routes.
It's fine.


Bluetooth: host: Implement L2CAP ecred reconfiguration request as initiator

Is your enhancement proposal related to a problem? Please describe.
Currently reconfiguaration request can only be received and not send. The problem is that this should be supported, as Zephyr declares Enhanced Credit Based Flow Control Channel Initiator feature to be supported (TSPC_L2CAP_1_5	LE Data Channel Initiator and TSPC_L2CAP_2_48 Support Enhanced Credit Based Flow Control Mode are supported). Lack of implemented functions and API don't allow us to pass L2CAP/ECFC/BV-22-C and L2CAP/ECFC/BV-24-C.
Describe the solution you'd like
Mechanism of initiating L2CAP Enhanced channel reconfiguration should be implemented (send request and process response)
Describe alternatives you've considered
I don't see any alternatives when support of functionality is declared.
Additional context
Red rectangle marks what should be implemented according to Test Specifiacation

Since this seems to be mostly for qualification purposes (at least until now no user reported need for being able to reconfigure channel) we can try make is simple and hide it under CONFIG_BT_TESTING.
So my proposal would be to add CONFIG_BT_L2CAP_ECRED_LAZY_RECONF (or whatever name fits) that would cause channels to be created with default MPS/MTU and have new test api int bt_l2cap_chan_reconfigure(struct bt_l2cap_chan *chan); that would allow to reconfigure tx MTU/MPS to whatever is set by tx endpoint.

L2CAP/ECFC/BV-22-C is fixed by https://github.com/zephyrproject-rtos/zephyr/pull/38639
L2CAP/ECFC/BV-24-C can be waived as IUT doesn't support on-demand MPS decreasing


Rename aur rpc to aur query

Could you rebase this to only include 144d1bd? I merged #579 already.

Thanks, rebased and included a small fix to its man page while I was on it.

:+1:


Tab bar, show / hide, styling options

Show / Hide
I am utilising the supersonic.ui.tabs.hide() and supersonic.ui.tabs.show() as described here - http://docs.appgyver.com/supersonic/guides/ui/native-components/tab-bar/#hide
How can I make the transitions instant? The fade looks really broken, as it fades to / from black and then the content jerks down / up in its place.
If the whole effect was instant (as it happens when I change view) it wouldn't look so bad, but currently really doesn't feel very slick. Any ideas? I see the same behavior in the kitchensink – http://www.appgyver.com/supersonic/kitchensink
Styling
I am trying to style the tab bar and really struggling to do something pretty simple. I want the whole tab to change background-color when that tab is selected, not just the marker-top and icons / text bits.
If I change tab-bar tab-bar-item:selected { then it only changes the color directly behind the icons, and it seems to ignore width and heights.
The tab divider height seems static.
Icon vs label placement also seems static. Would be nice to put them below / left / right of the tab label (especially on tablet devices).

Are you on iOS or Android? There's no options currently for the tab bar hide/show, so it's a valid feature request. Could you make a separate issue out of that?
The current native CSS implementation is a bit limited, so there are some awfully solid walls that you might run into. Sorry! I'm pretty sure that you can't change the whole tab bar background color via a single active tab.
One possible hack would be to use supersonic.ui.views.current.whenVisible and whenHidden to apply a CSS background for the tab bar that way.
I'm fairly sure tab label placement can't be currently changed, and the tab divider might also have a hardcoded height.


Automation for Python

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

Can one of the admins verify this patch?

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go


Tests for the new FileHandle API

This includes a correctness fix where NSError.isEqual(_:) was not comparing errors the way ObjC NSError does.

@swift-ci please test


Commitment Ended: success / congratulations

User Story / Job to be Done / Problem
As a user with an expired commitment, I want to see a confirmation that I successfully completed my commitment
Potential solution ideas

confetti 🎊

Basic celebratory flow is there, confetti went MIA I believe


Update worker-base image to Python 3.11.1

Need to do this before #1338 (which enforces Python 3.11.1) because the PR test runner image gcr.io/oss-vdb/ci uses the worker image.
gcr.io/oss-vdb/ci should probably be automatically rebuilt for PRs if the worker image changes...

Will merge this after this week's release in case something breaks


Update fido2-compatibility.md

Windows Version tested: is it 20H2 or 1904. I think 1904 can be removed.

@gaubert-ms : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks, looks good to me

#sign-off


RegisterAction is missing

This is needed for the transfer event in NEP-5 contracts

Apparently RegisterAction is python


Make opt-in strategy more performant

Reviewing the DEBUG logs the allstar.yaml file from the repo <my-org>/.allstar is being read on each execution for each repo. However, in the opt in strategy, scorecard still attempts to run disabled checks in repos which are not opted in. While no output is provided, for large GitHub organisation this means the time to obtain and refresh results is very long.
Proposal:

Review the .allstar/allstar.yaml as currently done.
Only analyse the repos which are opted in.
Once complete, re-read .allstar/allstar.yaml and continue.

Benefits:

Time is not wasted analysing repos and impressing output.
For large organisations with many repos, allow for performance tuning, as well as security output tuning.
For private orgs, app's backend could be run as a CRON-type GitHub Action without wasting GHA minutes.
Better performance from OpenSSF-hosted runners.

@jonhall54 Thanks for asking for help. Are you using the DO_NOTHING_ON_OPT_OUT option? More details here:
https://github.com/ossf/allstar/blob/3f9627a89e0c1305ec7c7748b573dc3cea17a248/pkg/config/operator/operator.go#L62-L69
https://github.com/ossf/allstar/blob/3f9627a89e0c1305ec7c7748b573dc3cea17a248/pkg/config/operator/operator.go#L134-L140
If not, try setting  DO_NOTHING_ON_OPT_OUT=true and see if that works for you. If you are setting that, then there is probably a bug in the Scorecard policy that I'll look into.

@jeffmendoza when running a allstar.yaml with the following:
optConfig:
 disableRepoOverride: true
 optOutStrategy: false
 optInRepos:
 - .allstar

With the environment variable DO_NOTHING_ON_OPT_OUT, it simply opts out of all the repos, including those opted in. This is also the case when running ./allstar -once -repo <org name>/allstar.

Yes, after adding the allstar.yaml to enable the bot overall, next add individual yaml files to turn on the individual policies. ex: https://github.com/jeffmendoza/dot-allstar-quickstart/blob/main/binary_artifacts.yaml to turn on the Binary Artifacts policy. The links to the  policy config definitions are here: https://github.com/ossf/allstar?tab=readme-ov-file#policies


Support for ROCM 6

It seems ROCM 5.6  kind of works, but it really requires too much back and forth to have everything working, the new Fedora 40 brings official ROCM support but starting in ROCM 6.
I am using this config from https://github.com/elixir-nx/xla/issues/63
Mix.install(
  [
    {:web_driver_client, "~> 0.2.0"},
    {:kino, "~> 0.12.3"},
    {:req, "~> 0.4.14"},
    {:erlexec, "~> 2.0"},
    {:nx, github: "elixir-nx/nx", sparse: "nx", override: true},
    {:exla, github: "elixir-nx/nx", sparse: "exla", override: true}
  ],
  system_env: %{
    "XLA_ARCHIVE_URL" =>
      "https://static.jonatanklosko.com/builds/0.6.0/xla_extension-x86_64-linux-gnu-rocm.tar.gz",
    "ROCM_PATH" => "/usr/lib64/rocm/"
  },
  config: [nx: [default_backend: {EXLA.Backend, client: :host}]]

I managed to find every pkgs it was asking for (this took a while of back and forth) until I reached this:
18:36:37.767 [warning] The on_load function for module Elixir.EXLA.NIF returned:
{:error,
 {:load_failed,
  ~c"Failed to load NIF library /home/ja/.cache/mix/installs/elixir-1.16.2-erts-14.2.5/f3927a87654a1bf097d7e31b6277a9f8/_build/dev/lib/exla/priv/libexla: 'librocblas.so.3: cannot open shared object file: No such file or directory'"}}

My guess is xla_extension needs to be built for rocm 7 (librocblas.s0.4), I tried to build it myself but the requirements are too way off the current system (gcc versions and so on)
Will be great if there were official xla binaries for different ROCM versions, as there are for CUDA.
I understand ROCM support is in low priority, but it is really nice for start in AI as it works nicely in linux

I am also trying to reproduce the build by using the provided dockerfiles, but I always get errors:
[3,765 / 6,478] Compiling mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp; 22s local ... (16 actions, 15 running)
ERROR: /app/.cache/xla_extension/xla-771e38178340cbaaef8ff20f44da5407c15092cb/xla/service/gpu/BUILD:1158:23: Compiling xla/service/gpu/cub_sort_kernel.cu.cc failed: (Exit 1): crosstool_wrapper_driver_is_not_gcc failed: error executing command (from target //xla/service/gpu:cub_sort_kernel_u32) external/local_config_rocm/crosstool/clang/bin/crosstool_wrapper_driver_is_not_gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer ... (remaining 100 arguments skipped)
clang++: warning: argument unused during compilation: '-fcuda-flush-denormals-to-zero' [-Wunused-command-line-argument]
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr41 = V_MOV_B32_dpp undef $vgpr41(tied-def 0), $vgpr4, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr4 = V_MOV_B32_dpp undef $vgpr4(tied-def 0), killed $vgpr3, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr3 = V_MOV_B32_dpp undef $vgpr3(tied-def 0), $vgpr2, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr101 = V_MOV_B32_dpp undef $vgpr101(tied-def 0), $vgpr99, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr98 = V_MOV_B32_dpp undef $vgpr98(tied-def 0), $vgpr96, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr101 = V_MOV_B32_dpp undef $vgpr101(tied-def 0), $vgpr99, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr98 = V_MOV_B32_dpp undef $vgpr98(tied-def 0), $vgpr96, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr42 = V_MOV_B32_dpp undef $vgpr42(tied-def 0), $vgpr8, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr101 = V_MOV_B32_dpp undef $vgpr101(tied-def 0), $vgpr99, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr98 = V_MOV_B32_dpp undef $vgpr98(tied-def 0), $vgpr96, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr101 = V_MOV_B32_dpp undef $vgpr101(tied-def 0), $vgpr99, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr98 = V_MOV_B32_dpp undef $vgpr98(tied-def 0), $vgpr96, 322, 15, 15, 0, implicit $exec
12 errors generated when compiling for gfx1036.
Target //xla/extension:xla_extension failed to build

Did you try building by setting the XLA revision as in https://github.com/elixir-nx/xla/issues/63#issuecomment-1844195261?
Setting up the right environment for building was an issue before, that's why we have the Dockerfile. I don't know about ROCM 6, my best bet would be on updating to newer XLA could fix the build, but that usually involves changes to EXLA too. I think it would be a good idea to update sometime soon anyway, but no guarantees.
You could perhaps use Docker with 5.6 for computations/experimentation altogether, though I get it's not very convenient.

@jalberto I updated to the latest XLA revision and EXLA main already uses that. I tried building with ROCm 5.7, but there were errors indicating that XLA already assumes 6.0 (using symbols defined in 6.0+). So I updated the Docker image and managed to successfully build with ROCm 6.0.
Please try XLA_ARCHIVE_URL=https://static.jonatanklosko.com/builds/0.7.0/xla_extension-x86_64-linux-gnu-rocm.tar.gz and nx/exla main. If it doesn't work, you can also try building locally.

thanks, @jonatanklosko will test and report back

@jonatanklosko sorry for the delay, now I have a different error:
: CommandLine Error: Option 'x86-disable-avoid-SFB' registered more than once!
LLVM ERROR: inconsistency in registered CommandLine options

@jalberto is it when loading the precompiled binary or during build?

@jonatanklosko in case it helps:
* Getting nx (https://github.com/elixir-nx/nx.git - origin/main)
remote: Enumerating objects: 22709, done.
remote: Counting objects: 100% (4025/4025), done.
remote: Compressing objects: 100% (780/780), done.
remote: Total 22709 (delta 3456), reused 3661 (delta 3202), pack-reused 18684
* Getting exla (https://github.com/elixir-nx/nx.git - origin/main)
remote: Enumerating objects: 22709, done.
remote: Counting objects: 100% (4047/4047), done.
remote: Compressing objects: 100% (776/776), done.
remote: Total 22709 (delta 3480), reused 3687 (delta 3228), pack-reused 18662
Resolving Hex dependencies...
Resolution completed in 0.126s
New:
  castore 1.0.7
  certifi 2.12.0
  complex 0.5.0
  elixir_make 0.8.4
  erlexec 2.0.6
  finch 0.18.0
  fss 0.1.1
  hackney 1.20.1
  hpax 0.2.0
  idna 6.1.1
  jason 1.4.1
  kino 0.12.3
  metrics 1.0.1
  mime 2.0.5
  mimerl 1.3.0
  mint 1.6.0
  nimble_options 1.1.1
  nimble_ownership 0.3.1
  nimble_pool 1.1.0
  parse_trans 3.4.1
  req 0.4.14
  ssl_verify_fun 1.1.7
  table 0.1.2
  telemetry 1.2.1
  tesla 1.9.0
  unicode_util_compat 0.7.0
  web_driver_client 0.2.0
  xla 0.7.0
* Getting web_driver_client (Hex package)
* Getting kino (Hex package)
* Getting req (Hex package)
* Getting erlexec (Hex package)
* Getting telemetry (Hex package)
* Getting xla (Hex package)
* Getting elixir_make (Hex package)
* Getting nimble_pool (Hex package)
* Getting complex (Hex package)
* Getting finch (Hex package)
* Getting jason (Hex package)
* Getting mime (Hex package)
* Getting nimble_ownership (Hex package)
* Getting castore (Hex package)
* Getting mint (Hex package)
* Getting nimble_options (Hex package)
* Getting hpax (Hex package)
* Getting fss (Hex package)
* Getting table (Hex package)
* Getting hackney (Hex package)
* Getting tesla (Hex package)
* Getting certifi (Hex package)
* Getting idna (Hex package)
* Getting metrics (Hex package)
* Getting mimerl (Hex package)
* Getting parse_trans (Hex package)
* Getting ssl_verify_fun (Hex package)
* Getting unicode_util_compat (Hex package)
==> table
Compiling 5 files (.ex)
Generated table app
==> mime
Compiling 1 file (.ex)
Generated mime app
==> nimble_options
Compiling 3 files (.ex)
Generated nimble_options app
===> Analyzing applications...
===> Compiling unicode_util_compat
===> Analyzing applications...
===> Compiling idna
===> Analyzing applications...
===> Compiling telemetry
==> jason
Compiling 10 files (.ex)
Generated jason app
==> hpax
Compiling 4 files (.ex)
Generated hpax app
===> Analyzing applications...
===> Compiling mimerl
==> ssl_verify_fun
Compiling 7 files (.erl)
Generated ssl_verify_fun app
==> fss
Compiling 4 files (.ex)
Generated fss app
==> complex
Compiling 2 files (.ex)
Generated complex app
==> nx
Compiling 35 files (.ex)
Generated nx app
==> kino
Compiling 47 files (.ex)
Generated kino app
===> Analyzing applications...
===> Compiling certifi
===> Analyzing applications...
===> Compiling parse_trans
==> nimble_pool
Compiling 2 files (.ex)
Generated nimble_pool app
===> Fetching rebar3_hex v7.0.7
===> Fetching hex_core v0.8.4
===> Fetching verl v1.1.1
===> Analyzing applications...
===> Compiling hex_core
===> Compiling verl
===> Compiling rebar3_hex
===> Fetching rebar3_ex_doc v0.2.22
===> Analyzing applications...
===> Compiling rebar3_ex_doc
make: Entering directory '/home/ja/.cache/mix/installs/elixir-1.16.2-erts-14.2.5/946037843196e7227084dde47bdabba6/deps/erlexec/c_src'
g++ -g -std=c++11 -finline-functions -Wall -DHAVE_PTRACE -MMD -DUSE_POLL=1 -O3 -DNDEBUG -DHAVE_SETRESUID -DHAVE_PIPE2   -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -I/home/ja/.local/share/mise/installs/erlang/26.2.5/lib/erl_interface-5.5.1/include  -c -o ei++.o ei++.cpp
g++ -g -std=c++11 -finline-functions -Wall -DHAVE_PTRACE -MMD -DUSE_POLL=1 -O3 -DNDEBUG -DHAVE_SETRESUID -DHAVE_PIPE2   -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -I/home/ja/.local/share/mise/installs/erlang/26.2.5/lib/erl_interface-5.5.1/include  -c -o exec.o exec.cpp
g++ -g -std=c++11 -finline-functions -Wall -DHAVE_PTRACE -MMD -DUSE_POLL=1 -O3 -DNDEBUG -DHAVE_SETRESUID -DHAVE_PIPE2   -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -I/home/ja/.local/share/mise/installs/erlang/26.2.5/lib/erl_interface-5.5.1/include  -c -o exec_impl.o exec_impl.cpp
mkdir -p /home/ja/.cache/mix/installs/elixir-1.16.2-erts-14.2.5/946037843196e7227084dde47bdabba6/deps/erlexec/priv/x86_64-redhat-linux/
mkdir -p "/home/ja/.cache/mix/installs/elixir-1.16.2-erts-14.2.5/946037843196e7227084dde47bdabba6/deps/erlexec/priv/x86_64-redhat-linux/"
g++ ei++.o exec.o exec_impl.o -L/home/ja/.local/share/mise/installs/erlang/26.2.5/lib/erl_interface-5.5.1/lib -lei -o /home/ja/.cache/mix/installs/elixir-1.16.2-erts-14.2.5/946037843196e7227084dde47bdabba6/deps/erlexec/priv/x86_64-redhat-linux/exec-port
make: Leaving directory '/home/ja/.cache/mix/installs/elixir-1.16.2-erts-14.2.5/946037843196e7227084dde47bdabba6/deps/erlexec/c_src'
===> Analyzing applications...
===> Compiling erlexec
===> Analyzing applications...
===> Compiling metrics
===> Analyzing applications...
===> Compiling hackney
==> castore
Compiling 1 file (.ex)
Generated castore app
==> elixir_make
Compiling 8 files (.ex)
Generated elixir_make app
==> xla
Compiling 2 files (.ex)
Generated xla app
==> exla
Unpacking /home/ja/.cache/xla/0.7.0/cache/external/xla_extension-4j534fd5eueir3oelhrj2pvadm.tar.gz into /home/ja/.cache/mix/installs/elixir-1.16.2-erts-14.2.5/946037843196e7227084dde47bdabba6/deps/exla/exla/cache
Using libexla.so from /home/ja/.cache/xla/exla/elixir-1.16.2-erts-14.2.5-xla-0.7.0-exla-0.7.1-4hm2i3sdtzvi2nwhnlfl4jx27u/libexla.so
g++ -fPIC -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -Icache/xla_extension/include -Wall -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers -Wno-comment -std=c++17 -w -DLLVM_VERSION_STRING= -O3 -c c_src/exla/exla.cc -o cache/objs/exla.o
g++ -fPIC -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -Icache/xla_extension/include -Wall -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers -Wno-comment -std=c++17 -w -DLLVM_VERSION_STRING= -O3 -c c_src/exla/exla_mlir.cc -o cache/objs/exla_mlir.o
g++ -fPIC -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -Icache/xla_extension/include -Wall -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers -Wno-comment -std=c++17 -w -DLLVM_VERSION_STRING= -O3 -c c_src/exla/custom_calls.cc -o cache/objs/custom_calls.o
g++ -fPIC -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -Icache/xla_extension/include -Wall -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers -Wno-comment -std=c++17 -w -DLLVM_VERSION_STRING= -O3 -c c_src/exla/exla_client.cc -o cache/objs/exla_client.o
g++ -fPIC -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -Icache/xla_extension/include -Wall -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers -Wno-comment -std=c++17 -w -DLLVM_VERSION_STRING= -O3 -c c_src/exla/exla_cuda.cc -o cache/objs/exla_cuda.o
g++ -fPIC -I/home/ja/.local/share/mise/installs/erlang/26.2.5/erts-14.2.5/include -Icache/xla_extension/include -Wall -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers -Wno-comment -std=c++17 -w -DLLVM_VERSION_STRING= -O3 -c c_src/exla/exla_nif_util.cc -o cache/objs/exla_nif_util.o
g++ cache/objs/exla.o cache/objs/exla_mlir.o cache/objs/custom_calls.o cache/objs/exla_client.o cache/objs/exla_nif_util.o cache/objs/exla_cuda.o -o cache/libexla.so -Lcache/xla_extension/lib -lxla_extension -shared -Wl,-rpath,'$ORIGIN/xla_extension/lib'
Compiling 23 files (.ex)

As a sanity check, try without XLA_ARCHIVE_URL, which by default should download just the CPU-enabled binary. This way we will know if it is specific to the ROCm binary. Make sure to reinstall without cache.

I see, I have no idea where this LLVM error is coming from, I didn't find x86-disable-avoid-SFB, nor X86AvoidStoreForwardingBlocks in openxla/xla source mentioned explicitly. You can try building youtself with XLA_BUILD=1 just in case, but that's a long shot (and provided that it builds without issues) :<

Not sure if this is completely related, but I'm trying to get ROCm 6 working too and built xla with the Dockerized build.sh script which gave me a tarball.
When I set XLA_ARCHIVE_PATH to https://s3.fr-par.scw.cloud/assets.stanko.io/hex/xla/rocm/0.8.0/xla_extension-x86_64-linux-gnu-rocm.tar.gz (which the tarball I built), Mix.install passes, but when something calls EXLA I get a unable to find ld.lld in PATH: No such file or directory error. I tried adding both ROCm bins and llvm bins to my PATH (export PATH="/opt/rocm/llvm/bin:/opt/rocm/bin:$PATH"), but I still get the same error.

Just wanted to ask if this is a known problem, or if someone has some pointers for debugging this?
Also, is there another way to use the built tarball except uploading it somewhere and setting XLA_ARCHIVE PATH?

@monorkin it may not be related, but the only thing I can think of is to also set export ROCM_PATH="/opt/rocm-6.0" (or whatever the version is).

Is there another way to use the built tarball except uploading it somewhere and setting XLA_ARCHIVE PATH?

I've just added support for XLA_ARCHIVE_PATH (#99), but that's going to be applicable only from the next release, so for now you need to use XLA_ARCHIVE_URL.

And are there plans to provide pre-built ROCm packages like for CUDA?

Not at the moment. The ROCm support is somewhat experimental, in the sense that we don't have the capacity to test it on every release and maintain possibly multiple precompiled builds. Jax (the Python library using XLA) also considers it experimental. This may change in the future, depending on how the ROCm prominence evolves upstream.


[Contact] File Open with dialog implement

Solved at https://github.com/namhokim/cocoa_app/pull/16


Fix many spelling errors

Fix various spelling errors that I found with codespell (and a few that I noticed myself). In a few cases, fix bad grammar, as well.
I left some errors unfixed in the cases where they were a part of the API (primarily ones involving "splitted"), where they were in generated code, or where I didn't fully understand the meaning of the sentence.
Motivation and context
Bad spelling hurts comprehension and looks unprofessional. :-)
How has this been tested?
I ran the cvat-core unit tests, but other than that, I'm relying on GitHub CI.
Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file seems too minor to warrant changelog inclusion
[ ] I have updated the documentation accordingly N/A, although some fixes are to the documentation, as well.
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[x] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@IRDonch , could you please look at linter errors? Looks like in the past we committed  these files with some problems.


Launching the BriefingBook web app raises alert message and does not load

Try to open https://nsec.esri.com/SolutionsWeb/Apps/NationalIntel/BriefingBook/ and the following error is thrown:
Please configure your URL for your ArcGIS Online Organization or Portal for ArcGIS site.

Briefing book config.js file had a syntax error in the "flickr":{}. Removed erroneous "api" text and briefing book opens.

Closing issue.

@nfeuerstein @conklinbd Also fixed this on the UC trial server.


Add more testing

Considering some people have problems with HTTP errors in production, adding some API and e2e tests might be a good idea.
These tests can be run in a container so Github action should be able to run these tests easily for each PR.
Postman can be used for API testing & cypress for e2e testing.

+1!


Adjust deprecated uses of include for Ansible 2.4.0+

Ansible 2.4.0 causes many examples to show deprecation warnings (with default Ansible settings), e.g.:
[DEPRECATION WARNING]: The use of 'include' for tasks has been deprecated. Use
'import_tasks' for static inclusions or 'include_tasks' for dynamic inclusions.
 This feature will be removed in a future release. Deprecation warnings can be
disabled by setting deprecation_warnings=False in ansible.cfg.
[DEPRECATION WARNING]: include is kept for backwards compatibility but usage is
 discouraged. The module documentation details page may explain more about this
 rationale.. This feature will be removed in a future release. Deprecation
warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.

It's slightly annoying that these warnings pop immediately in the first version where the new usage is possible—it would be nicer if people didn't start seeing the deprecation until a release or two after new usage is available, so I could transition book examples, roles, playbooks, etc. sooner, without having to worry about cutting off backwards compatibility with every previous Ansible version on day one :(
Anyways, these deprecated usages should be modified to use the new include/import techniques.

This has been completed after a series of fixes for various playbooks. Some roles still use include for compatibility reasons, but all of the code in this repository uses import_ or include_ tasks/playbooks now.


[Docs] Typo in pricing-fr.md

In documentation/src/docs/documentation/pricing-fr.md,
there's a typo at "doûte" (last paragraph).
I'm on my phone now, so I can't
PR, but this could be replaced by "doute".

@fr3nchalp4a Toutes mes excuses !
The fix is beeing deployed


Update/add multimodal test resources

Describe your changes
Add test_resources for multimodal tasks
Checklist before requesting a review

[x] I have performed a self-review of my code

Check the correct box. Does this PR contain:

[ ] Breaking changes
[x] New functionality
[ ] Major changes
[ ] Minor changes
[ ] Bug fixes

[x] Proposed changes are described in the CHANGELOG.md

[x] CI Tests succeed and look good!

I agree, I used an alternative in my test script which didn't seem to work. didn't know about the .A


Please make Dependency Injection more explicit for readability

I don't like the approach of "automagically" wiring a property, I would like control in this and have it be more explicit, so people of my team or people that take over my code know things are being injected at least, or know how a property is filled.

constructor injection is presently being worked on. might be able to
release tomorrow.
On Fri, Feb 25, 2022, 12:57 PM Yves Schelpe @.***>
wrote:

I don't like the approach of "automagically" wiring a property, I would
like control in this and have it be more explicit, so people of my team or
people that take over my code know things are being injected at least, or
know how a property is filled.
—
Reply to this email directly, view it on GitHub
https://github.com/dj-nitehawk/FastEndpoints/issues/43, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVXVOFENFBSLR6CEZULTADU44VPNANCNFSM5PJQFRZA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

v3.7.0 with constructor injection support ;-)


04/25/2018 03:12 : Create Pull Request for branch master -> prTesting1_2018042503

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Show full description of step when small step

For small steps, the "overflow : hidden" property prevents to see the caption properly. Could there be a way to either show a tooltip with a mouse over with the full description, or to choose wether or not the description should be "aligned" with the step size?

Step has now description field that is shown as a tooltip if set. Supports also HTML content in the same way as all tooltips in Vaadin components. Checkout latest version from master branch.

Cheers for that!


minor tweaks

Welcome. These changes will need some time to be updated and I'll add ASP.NET in front of MVC, and may change other related files!

@badersur Those sound like good changes. Let me know when it's ready for review again.

@scottaddie Could you take a look at my changes?

@badersur Thank you for cleaning this up for us!


Give actionable advice when rustdoc JSON files fail to parse.

Resolves #105.

@epage would you say this is also sufficient for #88 or did you have something else in mind for that one? I wasn't sure if information about which nightly ranges use which rustdoc format versions is available somewhere, for example.


Use icon (emoji) for file defined via icon: property

Is your feature request related to a problem? Please describe.
The plugin saves the icons for files (and folders) somewhere external to the Markdown files. Specifically icons for files is what this FR is about.
Describe the solution you'd like
If I could define icon: as a front matter property and give it an emoji, the plugin could use its value for a file's icon. icon: is a convention also used by Logseq (maybe the property name which defines the icon should be user-definable, e.g. iconize-icon:?). It'd be good to check that the property's value is actually an emoji.
Additional context
Having the file's icon defined in the Markdown would mean not losing such icon customisations if moving away from Obsidian or publishing with some other Markdown tool.
It's not clear whether if a file has been assigned an icon using the currently possible method, that it should be overridden if icon: exists within its Markdown, or vice versa, or to allow only one icon to be assigned to a file at a time. If icon: overrides, deleting the property would then have a fallback if the icon was also set using the current icon assigning method, which may or may not be what you want, but it sounds sane enough.
Thanks.

Thank you for the feature request. Might this be the same as #38?

It sounds similar, but it's not quite clear what that FR is (latter comments don't help much either). It appears to be more to do with defining a file's icon via some detectable keyword in the Markdown (front matter value or tag) which obsidian-iconize would then reference against its known list of icon names and apply to the file. It also doesn't store the icon itself in the Markdown file like my FR suggests to do via emojis.
In short, I see your plugin can display emoji next to file names and in tabs and I'd like to define that emoji per Markdown file via a YAML icon: property. I myself am not interested in leveraging any image icons for files. The emoji displayed would be the OS's native style.
If this FR were to be implemented, both YAML and JSON front matter should likely be supported due to Obsidian supporting both (upon property edits via the UI, Obsidian rewrites JSON front matter to YAML, so not supporting JSON "icon": isn't likely a big deal).

It sounds similar, but it's not quite clear what that FR is (latter comments don't help much either). It appears to be more to do with defining a file's icon via some detectable keyword in the Markdown (front matter value or tag) which obsidian-iconize would then reference against its known list of icon names and apply to the file. It also doesn't store the icon itself in the Markdown file like my FR suggests to do via emojis.
In short, I see your plugin can display emoji next to file names and in tabs and I'd like to define that emoji per Markdown file via a YAML icon: property. I myself am not interested in leveraging any image icons for files. The emoji displayed would be the OS's native style.
If this FR were to be implemented, reading the icon from both YAML and JSON front matter may need to be supported due to Obsidian supporting both (upon property edits via the UI, Obsidian rewrites JSON front matter to YAML anyway, so not supporting JSON "icon": likely isn't a big deal).

I got you. Your request is really specific to Emojis, if I am not mistaken. That's why I am unsure of implementing this feature. I'll think about it.

I got you. Your request is really specific to Emojis, if I am not mistaken. That's why I am unsure of implementing this feature. I'll think about it.

Perhaps a standalone plugin would better handle this, however it doesn't seem far removed to what this plugin already does: give symbols to things in Obsidian. Why not via an additional method? Thanks anyway.

Why not do something similar to Obsidian Banners, where if emojis are entered they are shown as Twemoji / current selected preference in the icon itself, but allow typing in the icon ids defined by custom icon packs like "LiFileText" to use those instead, and just automate the selection through the Icon UI?

The current workaround for the idea I have proposed in this FR is to rely on title: and use a plugin called Front Matter Title instead. It lets me assign a title: value, which I prepend with an emoji to set my "icon", and for the value to shown as the inline title, explorer list item title, graph node title, etc, without affecting the filename. In short, I can have an emoji set for my file which is then shown across Obsidian's UI wherever the filenames are displayed.
The title: value is also typically leveraged by software which can publish your notes online, so online versions would also have an emoji prepended to the displayed title (this benefit alone may make this approach better than the FR above, unless publishing tools can read and display icon: when set to an emoji too -- some read icon: but I believe valid values are only image paths).

Why not do something similar to Obsidian Banners, where if emojis are entered they are shown as Twemoji / current selected preference in the icon itself, but allow typing in the icon ids defined by custom icon packs like "LiFileText" to use those instead, and just automate the selection through the Icon UI?

That's what I am planning to implement as well (same for in text icons).
Regarding the front matter feature: I think this feature might be incomplete because there are still folders that cannot have a front matter property. So it might only apply to files, which I think, can be confusing to other users.

Would it be possible to include a Icon property as an optional override to the default functionality as mentioned by @redactedscribe in the original F.R?
The icon itself can be defined similar to #273, a defined icons property would override that notes icon, as mentioned originally. (also it'd be nice to have an option to automatically assign a property when selecting an Icon from the UI where applicable. (not for folders)

Having the Icons determinable in properties aligns with the Durability & Malleability obsidian aims to maintain within it's markdown files.
It also allows for the interoperability of plugins, for example, I can use Obsidian Banners if I prefer it's implementation of header Banners / Icons, and use Iconize functionality for everything else, like adding Icons and Tab Icons, while having them permantly embedded within my notes themselves if you are ever forced to stop developing this plugin due to burnout or any number of reasons.
I see where your coming from with it being confusing for some users, and I agree, obsidian really needs to add a way to make properties defined by plugins to be hidden so they dont clutter & confuse users, though, in the mean-time having it only auto-assign if you opt-in, and fundamentally being an override for the default functionality I feel resolves this,
For folders it shouldn't really be an option as folder properties would fundamentally have to be defined within obsidian itself (if they added them), and at that point it's no different to just being in Iconize's data.json, as it doesn't allow for the durability & independance of the .md files produced with vanilla obsidian which should be completely independant from both obsidian, in-case someone wanted to migrate to logSeq/others  (as said in the o.g. F.R by @redactedscribe)
Also folder properties wouldn't be accessible from other markdown based tooling, like publishing tools (also mentioned earlier)

Personal Anecdote being when publishing my notes to a Astro site, I can't access properties defined by Iconize, but can read those defined by Banners as they are defined in frontmatter.

Thanks for reading my dexy induced essay, theres every chance I missed some context but writing is more dopamine inducing then reading :)

Looks like this will be possible in v2.7.0 when #284 is fixed.

This has been resolved and added in >2.7.0! Cheers ☕

@FlorianWoelki I wanted to let you know that I've made a similar FR (https://github.com/snezhig/obsidian-front-matter-title/issues/182) for another plugin due to the fact it supports modifying the displayed title (via title:) in many other UI areas (bookmarks, graph, etc) compared to Iconize (only Files, tab, and note). I think supporting emoji via icon: is a good step for Iconize but I'm not sure how interested you'd be in supporting other UI areas which Front Matter Title (FMT) already does. The goal is to not duplicate developer efforts. Take a look if you have time.


Automate Critical CSS generation

https://nystudio107.com/blog/implementing-critical-css
Using this method and a templating language to generate front end templates implement automated critical CSS

See #27


how to get unread threads and mark messages as read

hi, i just want to know how to mark messages as read. i tried the markAsRead and markAsSeen but when i check on the facebook messenger it is still flagged and unread/unseen. thanks.
and how to get unread threads.
thanks

Related to #171. Just like markAsSeen is broken, so is markAsRead, sadly. I'm not sure how exactly to fix this yet, but if someone can come up with a fix, please do make a PR!

Oh, and about the unread threads, I don't think fbchat has that functionality yet, though it's probably possible somehow.

Any function to check unread messages from recent threads? Or just any method?

Not directly, no. You'd have to combine Client.fetchUnread and Client.fetchThreadMessages


merge from konsum-frontend-svelte

.github/workflows/browser_ci.yml

chore: add ${{ matrix.os }} (jobs.test.runs-on)
chore(deps): add 18.7.0 (jobs.test.strategy.matrix.node)

:tada: This PR is included in version 1.32.0 :tada:
The release is available on:

npm package (@latest dist-tag)
v1.32.0
GitHub release

Your semantic-release bot :package::rocket:


bundlerEnv does not match bundler gem to ruby version

Issue description
When building a ruby app based on bundlerEnv, the generated bin stub includes a reference to global bundler, which only includes bundler for the default ruby version.
$ grep bundler $(nix-build . -A rubocop)/bin/rubocop
Gem.use_paths("/nix/store/gz9z1zhwwvlz8fflx0rxvl8gqi8s6p7f-bundler-1.17.2/lib/ruby/gems/2.3.0", ENV["GEM_PATH"])

$ ls /nix/store/gz9z1zhwwvlz8fflx0rxvl8gqi8s6p7f-bundler-1.17.2/lib/ruby/gems/
2.5.0

Steps to reproduce
Modify the nix package for rubocop to call bundlerEnv with non-standard ruby, like bundlerEnv { ruby = ruby_2_3; }, and attempt to run the result. The following error is expected:
/nix/store/kyxxk7azqapja83v38mkgsnr5d6wg889-ruby-2.3.8/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require': cannot load such file -- bundler (LoadError)
	from /nix/store/kyxxk7azqapja83v38mkgsnr5d6wg889-ruby-2.3.8/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
	from ./result/bin/rubocop:15:in `<main>'

To make this work, something like this works:
diff --git a/pkgs/development/tools/rubocop/default.nix b/pkgs/development/tools/rubocop/default.nix
index 2b9792e4df2..3735d725bc2 100644
--- a/pkgs/development/tools/rubocop/default.nix
+++ b/pkgs/development/tools/rubocop/default.nix
@@ -1,9 +1,17 @@
-{ lib, bundlerEnv, ruby }:
+{ lib, bundlerEnv, bundler, ruby_2_3 }:

-bundlerEnv rec {
+let
+  bundlerEnv_ = bundlerEnv.override {
+    bundler = bundler.override {
+      ruby = ruby_2_3;
+    };
+  };
+in
+
+bundlerEnv_ rec {
   pname = "rubocop";

-  inherit ruby;
+  ruby = ruby_2_3;

   gemdir = ./.;

Technical details

system: "x86_64-linux"
host os: Linux 4.14.93, NixOS, 18.09.1922.97e0d53d669 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-18.09.1922.97e0d53d669"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

I had an issue with identical symptoms, however your solution didn’t work for me (I don’t exclude a misuse of bundlerEnv in my case though). After investigation, my issue is related to the change in https://github.com/NixOS/nixpkgs/pull/51842#issuecomment-478392799 (follow the link for details), which comes later than your current version, and that I solved with a "simple"
bundlerEnv {
  (...)
  ruby = ruby_2_4.overrideAttrs(old: {
    postInstall = builtins.replaceStrings [" --destdir $GEM_HOME"] [""] old.postInstall;
  })
}

If I’m right, this will be an issue in the same situation as you, that is whenever the ruby version is not the latest one.
I post it there so that people with similar issue can try both solutions.

@immae Thanks so much for sharing your workaround! This was the last blocker for us getting a proof-of-concept set up with ruby_2_3. I'd be happy to share a little sample project with a works.nix and broken.nix if that would help get this sort out in Nix 2.3